This is an archive of the discontinued LLVM Phabricator instance.

[test] Remove occurences of br undef in various llvm tests
AbandonedPublic

Authored by kritgpt on Mar 3 2023, 7:47 PM.

Diff Detail

Event Timeline

kritgpt created this revision.Mar 3 2023, 7:47 PM
Herald added a project: Restricted Project. · View Herald Transcript
kritgpt requested review of this revision.Mar 3 2023, 7:47 PM
kritgpt edited reviewers, added: nlopes; removed: ributzka, jdoerfert, sstefan1.Mar 3 2023, 7:49 PM
jrtc27 added a comment.Mar 3 2023, 7:56 PM

A cursory glance reveals numerous cases of ( i1, and cases of undefined variables (e.g. llvm/test/CodeGen/PowerPC/subreg-postra.ll).

Also, this is a *massive* patch. For something like this where I assume (I hope!) it's been done mechanically it would be best to provide the script used.

Not sure %replaceUndefN is good style for these tests either, don't we normally use snake_case for IR?

kritgpt abandoned this revision.Mar 9 2023, 4:28 AM

Splitting into multiple patches as suggested.