[PATCH] D15025: [ThinLTO] Option to invoke ThinLTO backend passes and importing

Mehdi AMINI via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 3 12:18:33 PST 2015


joker.eph added a comment.

Looks good to me but I'd like another opinion on the name of "-fthinlto-backend", Duncan?


================
Comment at: include/clang/Driver/Types.h:68
@@ -66,1 +67,3 @@
+  bool isLLVMIR(ID Id);
+
   /// isCuda - Is this a CUDA input.
----------------
I guess you could already commit this separately

================
Comment at: lib/CodeGen/CodeGenAction.cpp:819
@@ +818,3 @@
+                              llvm::Linker::Flags::None, Index.get()))
+        return;
+
----------------
I know, we can improve it later, I just wanted to share this so we're on the same page on the ultimate goal :)


http://reviews.llvm.org/D15025





More information about the cfe-commits mailing list