[PATCH] D15156: [ThinLTO] Appending linkage fixes

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 2 11:25:27 PST 2015


tejohnson added inline comments.

================
Comment at: test/Linker/funcimport_appending_global.ll:6
@@ +5,3 @@
+; Do the import now
+; RUN: llvm-link %t.bc -functionindex=%t3.thinlto.bc -import=foo:%t2.bc -S | FileCheck %s
+
----------------
rafael wrote:
> Can you just pass %s to llvm-link and remove the llvm-as run?
I can't because the .ll file doesn't have the function summary. Running llvm-as with the -function-summary option  is required to get that, then llvm-lto creates the combined function index to drive importing.


http://reviews.llvm.org/D15156





More information about the llvm-commits mailing list