[PATCH] D19636: [ThinLTO] Add option to emit imports files for distributed backends

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Wed May 4 16:14:57 PDT 2016


tejohnson added a comment.

thanks


================
Comment at: include/llvm/LTO/ThinLTOCodeGenerator.h:198
@@ -197,1 +197,3 @@
   /**
+   * Compute and emit the list of imported files for each module.
+   */
----------------
joker.eph wrote:
> for *each* module? I'd say for `\p ModulePath` instead.
Yes the old comment wasn't really correct, will fix

================
Comment at: include/llvm/LTO/ThinLTOCodeGenerator.h:201
@@ +200,3 @@
+  void emitImports(StringRef ModulePath, StringRef OutputName,
+                   ModuleSummaryIndex &Index);
+
----------------
joker.eph wrote:
> Same remark as the previous path: this does not really use anything specific to the ThinLTOCodeGenerator.
Will change this to a static method too.


http://reviews.llvm.org/D19636





More information about the llvm-commits mailing list