Hi Richard, On Thu, October 20, 2011 22:06, Richard Trieu wrote: > Update Clang to not complain when printing IntegerLiterals with type short, > unsigned short, __int128_t, or __uint128_t. This will fix PR11179. > > PR: > http://llvm.org/bugs/show_bug.cgi?id=11179 > > Patch also located at: > http://codereview.appspot.com/5309045/
For __int128_t and __uint128_t, the approach looks fine (though the printed value should include the "i128" or "Ui128" suffix). For short and unsigned short, it seems to me that this is a bug in Sema::BuildExpressionFromIntegralTemplateArgument: we shouldn't be creating IntegerLiteral AST nodes with those types (or with negative values, for that matter). Thanks, Richard _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
