Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-27 Thread Ollie Wild
On Thu, Apr 26, 2012 at 8:35 AM, Tom Tromey tro...@redhat.com wrote: This is ok with this change. Thanks. Updated and submitted to trunk. Ollie

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-26 Thread Tom Tromey
Ollie == Ollie Wild a...@google.com writes: Ollie 2012-04-22 Ollie Wild a...@google.com Ollie * gcc/c-family/c-common.c: Ollie * gcc/c-family/c-opts.c (c_common_handle_option): Ollie * gcc/c-family/c.opt: Ollie * gcc/doc/invoke.texi (struct A): Ollie *

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-26 Thread Paolo Bonzini
Il 23/04/2012 14:53, Gabriel Dos Reis ha scritto: I chose -Wreserved-user-defined-literal because that's the name used by Clang. It seemed better to maintain consistency between front ends than to invent a new, concise option name. Do you still want me to shorten in? yes. Why, out

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-25 Thread Ollie Wild
On Mon, Apr 23, 2012 at 8:28 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: Thanks.  Tom, I am satisfied with the diagnostic part. I think the CPP part makes sense but it is your call. Hi, Tom, just a quick checkin to see if you've had a chance to review this, yet. Thanks, Ollie

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-23 Thread Gabriel Dos Reis
On Sun, Apr 22, 2012 at 10:59 PM, Ollie Wild a...@google.com wrote: Add new option, -Wreserved-user-defined-literal. Just shorten it to -Wliteral-suffix. This option, which is enabled by default, causes the preprocessor to warn when a string or character literal is followed by a ud-suffix

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-23 Thread Ollie Wild
On Mon, Apr 23, 2012 at 7:10 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Sun, Apr 22, 2012 at 10:59 PM, Ollie Wild a...@google.com wrote: Add new option, -Wreserved-user-defined-literal. Just shorten it to -Wliteral-suffix. I chose -Wreserved-user-defined-literal because

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-23 Thread Gabriel Dos Reis
On Mon, Apr 23, 2012 at 7:30 AM, Ollie Wild a...@google.com wrote: On Mon, Apr 23, 2012 at 7:10 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Sun, Apr 22, 2012 at 10:59 PM, Ollie Wild a...@google.com wrote: Add new option, -Wreserved-user-defined-literal. Just shorten it to

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-23 Thread Ollie Wild
On Mon, Apr 23, 2012 at 7:53 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Mon, Apr 23, 2012 at 7:30 AM, Ollie Wild a...@google.com wrote: Do you still want me to shorten in? yes. Done. Updated patch attached. Ollie commit 3f53671fb7fc7811277f047e7914f78e127031a6 Author:

Re: PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-23 Thread Gabriel Dos Reis
On Mon, Apr 23, 2012 at 8:26 AM, Ollie Wild a...@google.com wrote: On Mon, Apr 23, 2012 at 7:53 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Mon, Apr 23, 2012 at 7:30 AM, Ollie Wild a...@google.com wrote: Do you still want me to shorten in? yes. Done.  Updated patch

PR c++/52538 Extend C++11 UDLs to be compatible with inttypes.h macros (issue6109043)

2012-04-22 Thread Ollie Wild
Add new option, -Wreserved-user-defined-literal. This option, which is enabled by default, causes the preprocessor to warn when a string or character literal is followed by a ud-suffix which does not begin with an underscore. According to [lex.ext]p10, this is ill-formed. Also modifies the