<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
<META NAME="GENERATOR" CONTENT="GtkHTML/4.6.6">
</HEAD>
<BODY>
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">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>
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>
</BODY>
</HTML>