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

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 17 13:43:28 PST 2015


rafael added a comment.

I got this warning:

/home/espindola/llvm/llvm/tools/gold/gold-plugin.cpp:133:17: warning: private field 'F' is not used [-Wunused-private-field]

  claimed_file *F


================
Comment at: tools/gold/gold-plugin.cpp:979
@@ +978,3 @@
+  // splitCodeGen will use a single thread since this has one entry.
+  std::vector<llvm::raw_pwrite_stream *> OSPtrs(1);
+  OSPtrs[0] = OS;
----------------
If you change codegenImpl to take an ArrayRef you don't have to do this.


http://reviews.llvm.org/D15390





More information about the llvm-commits mailing list