[PATCH] D17051: Get rid of CHECK-SAME-NOT in tests.

Jonathan Roelofs via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 9 15:55:03 PST 2016


jroelofs added a subscriber: jroelofs.

================
Comment at: test/Modules/ModuleDebugInfo.cpp:10
@@ -9,3 +9,3 @@
 // RUN: cat %t-mod.ll | FileCheck %s
 // RUN: cat %t-mod.ll | FileCheck --check-prefix=CHECK-NEG %s
 
----------------
While you're here, may as well shorten these three lines to:


```
// RUN: %clang_cc1 -triple %itanium_abi_triple -x objective-c++ -std=c++11 -debug-info-kind=limited -fmodules -fmodule-format=obj -fimplicit-module-maps -DMODULES -fmodules-cache-path=%t %s -I %S/Inputs -I %t -emit-llvm -o %t.ll -mllvm -debug-only=pchcontainer | FileCheck %s --check-prefix=CHECK --check-prefix=CHECK-NEG
```

(as long as you also move the one CHECK-NEG-NOT line up before all of the other `CHECK` lines)

================
Comment at: test/Modules/ModuleDebugInfo.cpp:25
@@ -25,1 +24,3 @@
+// CHECK-SAME:                    dwoId:
+// CHECK-SAME:                    )
 
----------------
This `CHECK-SAME` line and all the others are still dead. There's no `--check-prefix=CHECK-SAME`. This is really fishy.


http://reviews.llvm.org/D17051





More information about the cfe-commits mailing list