This is an archive of the discontinued LLVM Phabricator instance.

[X86] Use autogenerated result for switch-bt.ll test; NFC
ClosedPublic

Authored by goldstein.w.n on Jan 29 2023, 7:55 PM.

Diff Detail

Unit TestsFailed

Event Timeline

goldstein.w.n created this revision.Jan 29 2023, 7:55 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 29 2023, 7:55 PM
Herald added a subscriber: pengfei. · View Herald Transcript
goldstein.w.n requested review of this revision.Jan 29 2023, 7:55 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 29 2023, 7:55 PM
RKSimon added inline comments.Jan 30 2023, 1:36 AM
llvm/test/CodeGen/X86/switch-bt.ll
14

Don't these need removing as well?

258

can you move these helper comments someplace suitable (either in the switch statement for the 0/3/6 + 1/4/7 case groups or the test header comment)

goldstein.w.n marked 2 inline comments as done.Jan 30 2023, 9:44 AM
RKSimon accepted this revision.Jan 30 2023, 1:23 PM

LGTM

This revision is now accepted and ready to land.Jan 30 2023, 1:23 PM
This revision was landed with ongoing or failed builds.Jan 31 2023, 11:26 PM
This revision was automatically updated to reflect the committed changes.