This is an archive of the discontinued LLVM Phabricator instance.

[LSR][TTI][Arm] Enable isAllowDropLSRSolution for Arm
Needs ReviewPublic

Authored by eopXD on Nov 2 2022, 6:12 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

Depends on D136736. This patch is enables LSR drop solution when the solution
is less profitable than the baseline. The enablement shows regression on the
test case changed in this patch.

Please checkout https://reviews.llvm.org/D136736#3904001 for the LLVM IR after
the regressed example.

Diff Detail

Event Timeline

eopXD created this revision.Nov 2 2022, 6:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 2 2022, 6:12 PM
eopXD requested review of this revision.Nov 2 2022, 6:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 2 2022, 6:12 PM
eopXD edited the summary of this revision. (Show Details)Nov 2 2022, 6:14 PM
eopXD added a subscriber: fhahn.

@fhahn May you include someone more familiar to the Arm backend to this patch?

eopXD edited the summary of this revision. (Show Details)Nov 3 2022, 8:20 AM