This is an archive of the discontinued LLVM Phabricator instance.

[InstCombine] [NFC] use C.isNegatedPowerOf2() instead of (~C + 1).isPowerOf2()
ClosedPublic

Authored by Chenbing.Zheng on Jul 4 2022, 7:56 PM.

Diff Detail

Event Timeline

Chenbing.Zheng created this revision.Jul 4 2022, 7:56 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 4 2022, 7:56 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
Chenbing.Zheng requested review of this revision.Jul 4 2022, 7:56 PM
RKSimon accepted this revision.Jul 5 2022, 1:24 AM

LGTM - although its probably best that the test is committed separately as its not dependent on the NFC code change.

This revision is now accepted and ready to land.Jul 5 2022, 1:24 AM

LGTM - although its probably best that the test is committed separately as its not dependent on the NFC code change.

OK,I will separate it when I commit.

This revision was landed with ongoing or failed builds.Jul 5 2022, 2:05 AM
This revision was automatically updated to reflect the committed changes.