steven_wu added inline comments.

================
Comment at: lib/Driver/Tools.cpp:8326
-  if (C.getDriver().embedBitcodeEnabled() ||
-      C.getDriver().embedBitcodeMarkerOnly()) {
-    // Check if the toolchain supports bitcode build flow.
----------------
mehdi_amini wrote:
> Why aren't these tests returning true when LTO is enabled?
> I'm not sure why this is not the part that should be fixed instead.
These flags controls if there should be bitcode embedded in the object file. 
For the case of LTO, there is no object file thus there is no embedding.
If change the meaning of this flag, it will complicate the logic around line 
6355 and 4169. The other option is to add another flag but it seems little too 
much for passing one flag to ld.


https://reviews.llvm.org/D26690



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to