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

[Teresa Johnson via llvm-commits]

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