[PATCH] D45275: Add support for LTO options

Rumeet Dhindsa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 6 13:32:51 PDT 2018


rdhindsa 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
+
----------------
pcc wrote:
> 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.
Added new-pass-manager option to check if Module Pass manager is run.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D45275





More information about the llvm-commits mailing list