[PATCH] D86360: Add new hidden option -print-changed which only reports changes to IR

Jordan Rupprecht via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 3 10:20:13 PDT 2020


rupprecht added inline comments.


================
Comment at: llvm/test/Other/change-printer.ll:57
+; CHECK0: *** IR Pass PassManager<llvm::Function> (function: f) ignored ***
+; CHECK0: *** IR Pass ModuleToFunctionPassAdaptor<llvm::PassManager<llvm::Function> > (module) ignored ***
+; CHECK0: *** IR Dump After VerifierPass (module) omitted because no change ***
----------------
I'm seeing test failures that this should actually be: `ModuleToFunctionPassAdaptor<llvm::PassManager<llvm::Function>>` (note the lack of whitespace at the end in `>>`).

Is it safe to change it to that, or is this test somehow sensitive to the build/runtime environment? `> >` being required is a legacy of older versions of C++.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86360/new/

https://reviews.llvm.org/D86360



More information about the llvm-commits mailing list