Re: [PATCH, PR52654, C++11] Warn on overflow in user-defined literals

2012-11-28 Thread Jason Merrill
On 11/28/2012 06:13 PM, Ed Smith-Rowland wrote: Ultimately we should probably revisit this and 54413 by parsing the literal numbers in the C++ front end. That won't happen for 4.8. That sounds like you're proposing to duplicate code? - /* Both C and C++ require a diagnostic for a floating c

Re: [PATCH, PR52654, C++11] Warn on overflow in user-defined literals

2012-11-28 Thread Paolo Carlini
On 11/28/2012 04:12 AM, Ed Smith-Rowland wrote: Here is a final patch for PR52654 hopefully. Ultimately we should probably revisit this and 54413 by parsing the literal numbers in the C++ front end. That won't happen for 4.8. This builds and passes on x86_64-linux Thanks. Let's make sure Ja

[PATCH, PR52654, C++11] Warn on overflow in user-defined literals

2012-11-27 Thread Ed Smith-Rowland
Here is a final patch for PR52654 hopefully. Ultimately we should probably revisit this and 54413 by parsing the literal numbers in the C++ front end. That won't happen for 4.8. This builds and passes on x86_64-linux. Ed Index: gcc/c-family/c-lex.c