[PATCH] D28887: [PM] Make default pipeline test strict

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc added inline comments.


================
Comment at: test/Other/new-pm-defaults.ll:35-36
+;CHECK-O-NEXT: Running analysis: TargetLibraryAnalysis
+;CHECK-O-NEXT: Running pass: ModuleToFunctionPassAdaptor<llvm::PassManager<llvm::Function> >
+;CHECK-O-NEXT: Running analysis: InnerAnalysisManagerProxy<llvm::AnalysisManager<llvm::Function>, llvm::Module>
+;CHECK-O-NEXT: Starting llvm::Function pass manager run.
----------------
Here and below you'll need a bunch of {{.*}} and just dropping the tail of the pass name. You can see the patterns that work in the original CHECK lines.

The type printing is good, but not perfect across platforms.


================
Comment at: test/Other/new-pm-defaults.ll:147
+;CHECK-O-NEXT: Running pass: PrintModulePass
+;CHECK-O:      define void @foo(i32 %n) local_unnamed_addr {
+;CHECK-O-NEXT: entry:
----------------
Add some vertical space here for readability?


https://reviews.llvm.org/D28887