[PATCH] D15390: [ThinLTO] Launch importing backends in parallel threads from gold plugin

[Rafael Ávila de Espíndola via llvm-commits]

rafael added inline comments.

================
Comment at: test/tools/gold/X86/thinlto.ll:12
@@ -8,1 +11,3 @@
+; RUN:    --plugin-opt=thinlto \
+; RUN:    -shared %t.o %t2.o -o %t4
 
----------------
This gold invocation is not being tested.

================
Comment at: test/tools/gold/X86/thinlto.ll:32
@@ +31,3 @@
+; RUN:    -shared %t.o %t2.o -o %t4
+; RUN: llvm-bcanalyzer -dump %t4.thinlto.bc | FileCheck %s --check-prefix=COMBINED
+
----------------
These two only check the t4.thinlto.bc. Don't you want to, for example, run llvm-nm on t4?

================
Comment at: tools/gold/gold-plugin.cpp:33
@@ -32,2 +32,3 @@
 #include "llvm/Linker/IRMover.h"
+#include "llvm/Linker/Linker.h"
 #include "llvm/MC/SubtargetFeature.h"
----------------
Why do you need Linker.h?


http://reviews.llvm.org/D15390