<div class="gmail_quote">On Thu, Oct 20, 2011 at 2:53 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi Richard,<br>
<div><div class="h5"><br>
On Thu, October 20, 2011 22:06, Richard Trieu wrote:<br>
> Update Clang to not complain when printing IntegerLiterals with type short,<br>
> unsigned short, __int128_t, or __uint128_t.  This will fix PR11179.<br>
><br>
> PR:<br>
> <a href="http://llvm.org/bugs/show_bug.cgi?id=11179" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=11179</a><br>
><br>
> Patch also located at:<br>
> <a href="http://codereview.appspot.com/5309045/" target="_blank">http://codereview.appspot.com/5309045/</a><br>
<br>
</div></div>For __int128_t and __uint128_t, the approach looks fine (though the printed<br>
value should include the "i128" or "Ui128" suffix).</blockquote><div><br></div><div>Are those the correct suffixes?  Clang and gcc doesn't recognize them as valid integer constant suffixes.</div><div>
<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"> For short and unsigned<br>
short, it seems to me that this is a bug in<br>
Sema::BuildExpressionFromIntegralTemplateArgument: we shouldn't be creating<br>
IntegerLiteral AST nodes with those types (or with negative values, for that<br>
matter).<br></blockquote><div><br></div><div>Why can't an integer literal be short or negative?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<br>
Thanks,<br>
<span class="HOEnZb"><font color="#888888">Richard<br>
<br>
</font></span></blockquote></div><br>