[PATCH] D45275: Add support for LTO options

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 6 14:39:19 PDT 2018


pcc accepted this revision.
pcc added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lld/ELF/LTO.cpp:108-110
+  Conf.SampleProfile = Config->LTOSampleProfile;
+  Conf.UseNewPM = Config->LTONewPassManager;
+  Conf.DebugPassManager = Config->LTODebugPassManager;
----------------
Nit: I'd move these lines to above line 104 where the rest of the `Conf.foo = bar;` lines are.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D45275





More information about the llvm-commits mailing list