This is an archive of the discontinued LLVM Phabricator instance.

[ValueTracking] Add tests for getting knownbits from phi-edges; NFC
ClosedPublic

Authored by goldstein.w.n on Aug 12 2023, 2:43 PM.

Diff Detail

Event Timeline

goldstein.w.n created this revision.Aug 12 2023, 2:43 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 12 2023, 2:43 PM
goldstein.w.n requested review of this revision.Aug 12 2023, 2:43 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 12 2023, 2:43 PM
nikic added inline comments.Aug 14 2023, 3:01 AM
llvm/test/Analysis/ValueTracking/phi-known-bits.ll
24

Here again, I don't think the conditional branches are needed and just br label %T should do.

goldstein.w.n marked an inline comment as done.Aug 14 2023, 9:04 AM

Fixup br, add a few more tests

nikic accepted this revision.Aug 15 2023, 2:05 AM

LGTM

This revision is now accepted and ready to land.Aug 15 2023, 2:05 AM
This revision was landed with ongoing or failed builds.Aug 22 2023, 8:59 AM
This revision was automatically updated to reflect the committed changes.