[PATCH] D29065: Split isUsingLTO() outside of embedBitcodeInObject() and embedBitcodeMarkerOnly().

Mehdi AMINI via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 27 22:18:40 PST 2017


mehdi_amini added inline comments.


================
Comment at: cfe/trunk/lib/Driver/Tools.cpp:6464
   if (C.getDriver().isSaveTempsEnabled() &&
-      !C.getDriver().embedBitcodeInObject() && isa<CompileJobAction>(JA))
+      !C.getDriver().embedBitcodeInObject() && !C.getDriver().isUsingLTO() &&
+      isa<CompileJobAction>(JA))
----------------
steven_wu wrote:
> Should this line actually be:
> 
> ```
> !(C.getDriver().embedBitcodeInObject() && !C.getDriver().isUsingLTO()) &&
> 
> ```
> 
Thanks, r293370


Repository:
  rL LLVM

https://reviews.llvm.org/D29065





More information about the cfe-commits mailing list