Skip to content

[arm64] JIT: Fold "A * B + C" to MADD/MSUB #61037

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
EgorBo merged 9 commits into from
Nov 2, 2021
Merged

[arm64] JIT: Fold "A * B + C" to MADD/MSUB #61037

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
74bd53d
Fold "A * B + C" to madd
EgorBo Oct 30, 2021
eb100e1
fix arm32
EgorBo Oct 30, 2021
046cd22
guard with OptimizationEnabled
EgorBo Oct 30, 2021
c7a9da9
clean up
EgorBo Oct 31, 2021
5b1bc8f
Clean up
EgorBo Nov 1, 2021
40299d6
formatting
EgorBo Nov 1, 2021
7bf486d
Formatting
EgorBo Nov 1, 2021
8ae926f
Address feedback
EgorBo Nov 2, 2021
2591841
add assert
EgorBo Nov 2, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Formatting
  • Loading branch information
@EgorBo
EgorBo committed Nov 1, 2021
commit 7bf486d6f33fcd45401c0e171ac1f1df490f36cf
6 changes: 4 additions & 2 deletions src/coreclr/jit/lsrabuild.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3070,13 +3070,15 @@ int LinearScan::BuildOperandUses(GenTree* node, regMaskTP candidates)
BuildUse(node->gtGetOp1(), candidates);
return 1;
}
#endif // FEATURE_HW_INTRINSICS
if (node->OperIs(GT_MUL)) // Can be contained for MultiplyAdd
#endif // FEATURE_HW_INTRINSICS
if (node->OperIs(GT_MUL))
{
// Can be contained for MultiplyAdd
return BuildBinaryUses(node->AsOp(), candidates);
}
if (node->OperIs(GT_NEG))
{
// Can be contained for MultiplyAdd
return BuildOperandUses(node->gtGetOp1(), candidates);
}

Expand Down