This is an archive of the discontinued LLVM Phabricator instance.

[TargetLowering] make helper function for SetCC + and optimizations (NFC)
ClosedPublic

Authored by spatel on May 7 2016, 10:16 AM.

Details

Summary

After looking at D19087 again, it occurred to me that we can do better. If we consolidate the valueHasExactlyOneBitSet() transforms, we won't incur extra overhead from calling it a 2nd time, and we can shrink SimplifySetCC() a bit. No functional change intended.

Diff Detail

Event Timeline

spatel updated this revision to Diff 56502.May 7 2016, 10:16 AM
spatel retitled this revision from to [TargetLowering] make helper function for SetCC + and optimizations (NFC).
spatel updated this object.
spatel added reviewers: kbsmith1, hfinkel, andreadb.
spatel added a subscriber: llvm-commits.
kbsmith1 accepted this revision.May 9 2016, 9:31 AM
kbsmith1 edited edge metadata.

LGTM.

This revision is now accepted and ready to land.May 9 2016, 9:31 AM
This revision was automatically updated to reflect the committed changes.