<div dir="ltr">Thanks for fixing. +binji, original author of r211657.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 14, 2014 at 5:24 AM, Justin Holewinski <span dir="ltr"><<a href="mailto:jholewinski@nvidia.com" target="_blank">jholewinski@nvidia.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><u></u>


  
  

<div><div><div class="h5">
On Sun, 2014-07-13 at 16:42 -0700, Richard Smith wrote:
<blockquote type="CITE">
    On Sun, Jul 13, 2014 at 3:10 PM, Joerg Sonnenberger <<a href="mailto:joerg@britannica.bec.de" target="_blank">joerg@britannica.bec.de</a>> wrote:<br>
    <blockquote>
        Hi all,<br>
        the attached patch ensures that __INTPTR_TYPE__ and __UINTPTR_TYPE__ are<br>
        consistent as the existing logic would pick e.g. long and unsigned int<br>
        on ARM. One issue I found during this is that the NVPTX backend is<br>
        using unsigned types here, which doesn't make sense to me.
    </blockquote>
</blockquote>
<blockquote type="CITE">
    <br>
    <br>
</blockquote>
<blockquote type="CITE">
    The change to SemaOverload.cpp looks unrelated? Other than that, this LGTM, although I'm a little concerned that it might be intptr_t we were getting wrong on some of those targets rather than uintptr_t. Have you checked that this gives the same results as GCC?<br>

</blockquote>
<br></div></div>
There is no real reason we are using unsigned types for NVPTX, AFAIK.  Feel free to change it for consistency if needed.

<div>
<hr>
</div>
<div>This email message is for the sole use of the intended recipient(s) and may 
contain confidential information.  Any unauthorized review, use, disclosure 
or distribution is prohibited.  If you are not the intended recipient, 
please contact the sender by reply email and destroy all copies of the original 
message. </div>
<div>
<hr>
</div>
</div>

<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>