[Reassociate][test] Delete improper -NOT patterns to work with -enable-new-pm=1
authorFangrui Song <i@maskray.me>
Thu, 29 Oct 2020 06:58:31 +0000 (23:58 -0700)
committerFangrui Song <i@maskray.me>
Thu, 29 Oct 2020 06:59:11 +0000 (23:59 -0700)
The two tests rely on LegacyInlinerBase::doFinalization to remove
Function::isDefTriviallyDead() functions. The new PM does not have the behavior.
The -NEXT patterns checking the emptiness are actually sufficient.

Note, reassociate-deadinst.ll has become stale - it no longer catches
the problem r285380 intended. Unfortunately it is difficult to craft a
new test because it is actually pretty difficult to break it with
`MadeChange = true;` all over the file.

llvm/test/Transforms/Reassociate/erase_inst_made_change.ll
llvm/test/Transforms/Reassociate/reassociate-deadinst.ll

index febb944..0bb926f 100644 (file)
@@ -21,7 +21,6 @@ entry:
 }
 
 define internal void @bar() noinline nounwind readnone {
-; CHECK-NOT: bar
 entry:
   ret void
 }
index 9266fce..5384a65 100644 (file)
@@ -1,6 +1,5 @@
 ; RUN: opt < %s -inline -function-attrs -reassociate -S | FileCheck %s
 
-; CHECK-NOT: func1
 ; CHECK-LABEL: main
 ; CHECK-NEXT: ret void