[llvm] 4464d90 - [NFC][Reduce] Rewrite remove-funcs.ll to use FileCheck, make it less fragile

Roman Lebedev via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 22 06:12:31 PDT 2020


Author: Roman Lebedev
Date: 2020-07-22T16:11:56+03:00
New Revision: 4464d90ec6e1c9b26f5ef6edb9088c219b0647a1

URL: https://github.com/llvm/llvm-project/commit/4464d90ec6e1c9b26f5ef6edb9088c219b0647a1
DIFF: https://github.com/llvm/llvm-project/commit/4464d90ec6e1c9b26f5ef6edb9088c219b0647a1.diff

LOG: [NFC][Reduce] Rewrite remove-funcs.ll to use FileCheck, make it less fragile

Added: 
    

Modified: 
    llvm/test/Reduce/remove-funcs.ll

Removed: 
    llvm/test/Reduce/Inputs/remove-funcs.py


################################################################################
diff  --git a/llvm/test/Reduce/Inputs/remove-funcs.py b/llvm/test/Reduce/Inputs/remove-funcs.py
deleted file mode 100755
index 1d726c4e9ee7..000000000000
--- a/llvm/test/Reduce/Inputs/remove-funcs.py
+++ /dev/null
@@ -1,9 +0,0 @@
-
-import sys
-
-input = open(sys.argv[1], "r")
-for line in input:
-  if "@interesting" in line:
-    sys.exit(0)
-
-sys.exit(1) # IR isn't interesting

diff  --git a/llvm/test/Reduce/remove-funcs.ll b/llvm/test/Reduce/remove-funcs.ll
index 59ffd849193d..a9b525de0372 100644
--- a/llvm/test/Reduce/remove-funcs.ll
+++ b/llvm/test/Reduce/remove-funcs.ll
@@ -1,23 +1,27 @@
 ; Test that llvm-reduce can remove uninteresting functions as well as
 ; their InstCalls.
 ;
-; RUN: llvm-reduce --test %python --test-arg %p/Inputs/remove-funcs.py %s -o %t
-; RUN: cat %t | FileCheck -implicit-check-not=uninteresting %s
+; RUN: llvm-reduce --test FileCheck --test-arg --check-prefixes=CHECK-ALL,CHECK-INTERESTINGNESS --test-arg %s --test-arg --input-file %s -o %t
+; RUN: cat %t | FileCheck -implicit-check-not=uninteresting --check-prefixes=CHECK-ALL,CHECK-FINAL %s
 
 define i32 @uninteresting1() {
 entry:
   ret i32 0
 }
 
-; CHECK: interesting()
+; CHECK-ALL-LABEL: interesting()
 define i32 @interesting() {
 entry:
-  ; CHECK: call i32 @interesting()
+  ; CHECK-INTERESTINGNESS: call i32 @interesting()
   %call2 = call i32 @interesting()
   %call = call i32 @uninteresting1()
   ret i32 5
 }
 
+; CHECK-FINAL-NEXT: entry:
+; CHECK-FINAL-NEXT:   %call2 = call i32 @interesting()
+; CHECK-FINAL-NEXT: }
+
 define i32 @uninteresting2() {
 entry:
   ret i32 0


        


More information about the llvm-commits mailing list