This is an archive of the discontinued LLVM Phabricator instance.

[AArch64] Avoid materializing constant 1 by using csinc, rather than csel
ClosedPublic

Authored by mcrosier on Aug 25 2016, 2:25 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

mcrosier updated this revision to Diff 69291.Aug 25 2016, 2:25 PM
mcrosier retitled this revision from to [AArch64] Avoid materializing constant 1 by using csinc, rather than csel.
mcrosier updated this object.
mcrosier added reviewers: gberry, jmolloy, efriedma.
mcrosier added a subscriber: llvm-commits.
efriedma accepted this revision.Aug 25 2016, 2:41 PM
efriedma edited edge metadata.

LGTM. (And I guess that answers one of my questions about your other patch.)

This revision is now accepted and ready to land.Aug 25 2016, 2:41 PM

Yes, your feedback prompted this work. Thanks, Eli.

This revision was automatically updated to reflect the committed changes.