This is an archive of the discontinued LLVM Phabricator instance.

[test] Remove occurences of br undef in CodeGen/AMDGPU tests
ClosedPublic

Authored by kritgpt on Mar 8 2023, 3:39 PM.

Diff Detail

Event Timeline

kritgpt created this revision.Mar 8 2023, 3:39 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 8 2023, 3:39 PM
kritgpt requested review of this revision.Mar 8 2023, 3:39 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 8 2023, 3:39 PM
nlopes requested changes to this revision.Mar 9 2023, 12:05 AM

tests failing

This revision now requires changes to proceed.Mar 9 2023, 12:05 AM
kritgpt updated this revision to Diff 510736.Apr 4 2023, 3:43 AM

Split naive cases from files that need manual checking.

Patch now contains only naive cases.

nlopes accepted this revision.Apr 4 2023, 5:10 AM

LGTM

This revision is now accepted and ready to land.Apr 4 2023, 5:10 AM
This revision was landed with ongoing or failed builds.Apr 4 2023, 5:15 AM
This revision was automatically updated to reflect the committed changes.
nlopes added a comment.Apr 4 2023, 5:48 AM

This broke the build. I had to revert the patch.
You need to rebase your patch to latest git main & update the wave32.ll test.