<div dir="ltr">Yes, please. Sorry for the confusion. I updated my display name. Thanks</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 2, 2014 at 1:56 AM, Sahasrabuddhe, Sameer <span dir="ltr"><<a href="mailto:sameer.sahasrabuddhe@amd.com" target="_blank">sameer.sahasrabuddhe@amd.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 11/27/2014 8:12 AM, Ribulous Barnulous wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
Great. Anything else I need to do to get committed?<br>
<br></span>
On Tue, Nov 25, 2014 at 11:54 PM, Sahasrabuddhe, Sameer <<a href="mailto:sameer.sahasrabuddhe@amd.com" target="_blank">sameer.sahasrabuddhe@amd.com</a> <mailto:<a href="mailto:sameer.sahasrabuddhe@amd.com" target="_blank">sameer.sahasrabuddhe@<u></u>amd.com</a>>> wrote:<br>
<br>
    Thanks. LGTM!<br>
<br>
</blockquote>
<br>
Since nobody has submitted the patch so far, I could take a stab at it. Please holler if that is unexpected!<br>
<br>
I have commit access to the svn repo, which was originally meant for submitting minor fixes to Polly. I never needed to submit anything so this would be my first time. I checked out llvm trunk and cfe trunk, and then ran "ninja check-all" with the patch applied. Is there any other sanity check to be done?<br>
<br>
The commit message will be:<br>
<msg><br>
Always emit kernel arg info for SPIR.<br>
<br>
<a href="http://llvm.org/bugs/show_bug.cgi?id=21555" target="_blank">http://llvm.org/bugs/show_bug.<u></u>cgi?id=21555</a><span class=""><br>
<br>
Currently, kernel argument metadata is omitted unless the<br></span>
"-cl-kernel-arg-info" option is specified. But the SPIR 1.2 spec<br>
requires that all metadata except kernel_arg_name should always be<span class=""><br>
emitted, and kernel_arg_name is only emitted when<br>
"-cl-kernel-arg-info" is specified.<br>
<br></span>
Patch contributed by Ribulous Barnulous.<br>
</msg><br>
<br>
BTW, is "Ribulous Barnulous" the same as "Ryan Burn" from the bug? If so, which name should go in the commit?<span class="HOEnZb"><font color="#888888"><br>
<br>
Sameer.<br>
<br>
<br>
</font></span></blockquote></div><br></div>