<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 6, 2015 at 8:44 AM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">LGTM<br>
<br>
[assuming this just copies stuff from <a href="http://reviews.llvm.org/D8463" target="_blank">http://reviews.llvm.org/D8463</a> w/o modifications]<br>
<br></blockquote><div><br></div><div>The only change is that generated code uses '__' prefix for symbols it creates instead of '.' used in <a href="http://reviews.llvm.org/D8463" target="_blank">D8463</a>. No functional changes otherwise.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
================<br>
Comment at: lib/CodeGen/CGCUDANV.cpp:43<br>
@@ +42,3 @@<br>
+  llvm::SmallVector<llvm::Function *, 16> EmittedKernels;<br>
+  /// Keeps track of variables containing handles of GPU binaries Populated by<br>
+  /// ModuleCtorFunction() and used to create corresponding cleanup calls in<br>
----------------<br>
Nit: period (or semicolon) after "binaries"?<br></blockquote><div><br></div><div>Dot was missing. Fixed.</div><div><br></div><div>--Artem</div></div>
</div></div>