Page MenuHomePhabricator

[AArch64][GlobalISel] Add identity combines to post-legal combiner.

Authored by aemerson on Jul 26 2021, 11:50 AM.



We see some shifts of zero emitted during legalization.

Diff Detail

Event Timeline

aemerson created this revision.Jul 26 2021, 11:50 AM
aemerson requested review of this revision.Jul 26 2021, 11:50 AM
aemerson updated this revision to Diff 361743.Jul 26 2021, 11:54 AM

Update a test.

jroelofs accepted this revision.Jul 26 2021, 11:55 AM
jroelofs added inline comments.

Maybe the LHS ought to be a copy from a phys reg, so this test doesn't get accidentally repurposed if someone adds a constant-folding combines to it.

This revision is now accepted and ready to land.Jul 26 2021, 11:55 AM
aemerson added inline comments.Jul 26 2021, 11:59 AM

Good point.

aemerson updated this revision to Diff 361745.Jul 26 2021, 12:00 PM
This revision was landed with ongoing or failed builds.Jul 26 2021, 3:18 PM
This revision was automatically updated to reflect the committed changes.