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

Steven Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 27 19:18:19 PST 2017


steven_wu added a comment.

Sorry I just notice that when I look at driver today.



================
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))
----------------
Should this line actually be:

```
!(C.getDriver().embedBitcodeInObject() && !C.getDriver().isUsingLTO()) &&

```



Repository:
  rL LLVM

https://reviews.llvm.org/D29065





More information about the cfe-commits mailing list