[PATCH] D72538: [ThinLTO] Add additional ThinLTO pipeline testing with new PM

Wei Mi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 10 14:53:03 PST 2020


wmi added a comment.

The additional pipeline testing will catch any future pass change to the pipeline. A related but separate question is do we have a way to check whether there is any other missing pass in thinlto newpm similar as that in D72386 <https://reviews.llvm.org/D72386>?



================
Comment at: clang/test/CodeGen/thinlto-distributed-newpm.ll:11
+
+; RUN: %clang_cc1 -triple x86_64-grtev4-linux-gnu \
+; RUN:   -O2 -fexperimental-new-pass-manager -fdebug-pass-manager \
----------------
Can we test %clang intead of %clang_cc1? so we don't have to add flags like -vectorize-slp and -vectorize-loops. clang `O2/O3` pipeline is something we care about. Those flags added by clang to cc1 options may be subject to change.  



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72538





More information about the llvm-commits mailing list