[PATCH] D45275: Add support for LTO options

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 5 15:38:59 PDT 2018


pcc added inline comments.


================
Comment at: lld/test/ELF/lto/new-pass-manager.ll:7
+; RUN: ld.lld --plugin-opt=new-pass-manager %t.o -o %t.s.o
+; RUN: ld.lld --plugin-opt=thinlto --lto-new-pass-manager %t.o -o %t1.o
+
----------------
You don't actually need to pass `--plugin-opt=thinlto` here. The flag has no effect.

I would also do something like D45293 to make sure that we are actually using the new pass manager.


https://reviews.llvm.org/D45275





More information about the llvm-commits mailing list