[PATCH] D11908: Clang support for -fthinlto.

Teresa Johnson via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 13 10:24:51 PDT 2015

tejohnson added inline comments.

Comment at: include/clang/Driver/Options.td:685
@@ -684,1 +684,3 @@
 def fno_lto : Flag<["-"], "fno-lto">, Group<f_Group>;
+def fthinlto : Flag<["-"], "fthinlto">, Flags<[CC1Option]>, Group<f_Group>;
+def fno_thinlto : Flag<["-"], "fno-thinlto">, Group<f_Group>;
davidxl wrote:
> spell it like "fthin-lto" seems better than one word "fthinlto"
I considered that too, but wanted to distinguish from regular LTO. I could go either way.

Comment at: lib/CodeGen/BackendUtil.cpp:621
@@ -620,2 +620,3 @@
-        createBitcodeWriterPass(*OS, CodeGenOpts.EmitLLVMUseLists));
+        createBitcodeWriterPass(*OS, CodeGenOpts.EmitLLVMUseLists,
+                                CodeGenOpts.EmitThinLTOIndex));
davidxl wrote:
> The change to this function is missing in the patch.
Right, that is in LLVM and needs to be in a different patch (goes into a different repository). As mentioned in the patch summary this is dependent on LLVM patch http://reviews.llvm.org/D11907.


More information about the cfe-commits mailing list