[PATCH] D28997: [LTO] Teach lib/LTO about the new pass manager

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 22 18:01:06 PST 2017

davide added inline comments.

Comment at: lib/LTO/LTOBackend.cpp:248
+  if (Conf.OptUseNewPM || !Conf.OptPipeline.empty()) {
     // FIXME: Plumb the combined index into the new pass manager.
+    if (Conf.OptPipeline.empty())
mehdi_amini wrote:
> In the meantime, we shouldn't accept the combination ThinLTO+Conf.OptUseNewPM.
Indeed. Are you fine with this being a fatal error for the time being as the configuration is forbidden (similarly to what we do around line 212)?
Otherwise I can just change the return value to std::pair<Error, bool> or return Error and pass bool as argument by reference, or whatever you like better.



More information about the llvm-commits mailing list