This is an archive of the discontinued LLVM Phabricator instance.

[ValueTracking] Add tests for knownbits of `abs`; NFC
ClosedPublic

Authored by goldstein.w.n on May 7 2023, 11:58 PM.

Diff Detail

Event Timeline

goldstein.w.n created this revision.May 7 2023, 11:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 7 2023, 11:58 PM
Herald added a subscriber: StephenFan. · View Herald Transcript
goldstein.w.n requested review of this revision.May 7 2023, 11:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 7 2023, 11:58 PM
RKSimon accepted this revision.May 8 2023, 3:29 AM

LGTM - we already have the exhaustive tests in KnownBitsTest, but adding explicit test patterns for specific cases is fine.

This revision is now accepted and ready to land.May 8 2023, 3:29 AM

LGTM - we already have the exhaustive tests in KnownBitsTest, but adding explicit test patterns for specific cases is fine.

I see there are two things we want to test. 1) that the knownbits transformations are correct. 2) that is behaves as is expected in valuetracking. Without case 2 we could have a bug that just set all unknown.

This revision was landed with ongoing or failed builds.May 16 2023, 5:00 PM
This revision was automatically updated to reflect the committed changes.