<div dir="ltr">Committed in r206302, and the test in r206303.<div><br></div><div><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 14, 2014 at 5:41 PM, Manjunath Kudlur <span dir="ltr"><<a href="mailto:keveman@gmail.com" target="_blank">keveman@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><br>
<br>
================<br>
Comment at: lib/CodeGen/TargetInfo.cpp:4770<br>
@@ -4768,1 +4769,3 @@<br>
+  static void addNVVMMetadata(llvm::Function *F, StringRef Name,<br>
+                              const int Operand);<br>
 };<br>
----------------<br>
</div><div class="">Reid Kleckner wrote:<br>
> Having const ints feels silly, since the caller really doesn't care.<br>
</div>const removed.<br>
<div class=""><br>
================<br>
Comment at: lib/CodeGen/TargetInfo.cpp:4848<br>
@@ +4847,3 @@<br>
+      addNVVMMetadata(F, "maxntidx",<br>
+                       FD->getAttr<CUDALaunchBoundsAttr>()->getMaxThreads());<br>
+      // min blocks is a default argument for CUDALaunchBoundsAttr, so getting a<br>
----------------<br>
</div>Reid Kleckner wrote:<br>
> nit: indentation<br>
Fixed.<br>
<div class=""><br>
================<br>
Comment at: lib/CodeGen/TargetInfo.cpp:4853<br>
@@ +4852,3 @@<br>
+      // don't have to add a PTX directive.<br>
+      int minctasm = FD->getAttr<CUDALaunchBoundsAttr>()->getMinBlocks();<br>
+      if (minctasm > 0) {<br>
----------------<br>
</div><div class="">Reid Kleckner wrote:<br>
> The naming for local variables is StudlyCaps.<br>
</div>Fixed.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D3318" target="_blank">http://reviews.llvm.org/D3318</a><br>
<br>
<br>
</blockquote></div><br></div></div></div>