<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Jul 13, 2014 at 3:10 PM, Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@britannica.bec.de" target="_blank">joerg@britannica.bec.de</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">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.<br></blockquote><div><br></div><div>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?</div>
</div></div></div>