Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(19)

Issue 6104051: [google/integration] Extend C++11 UDLs to be compatible with inttypes.h macros

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 5 months ago by Ollie Wild
Modified:
9 years, 5 months ago
CC:
gcc-patches_gcc.gnu.org
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+108 lines, -8 lines) Patch
M gcc/c-family/c.opt View 1 chunk +4 lines, -0 lines 0 comments Download
M gcc/c-family/c-common.c View 1 chunk +1 line, -0 lines 0 comments Download
M gcc/c-family/c-opts.c View 1 chunk +4 lines, -0 lines 0 comments Download
gcc/doc/invoke.texi View 2 chunks +25 lines, -1 line 0 comments Download
A gcc/testsuite/g++.dg/cpp0x/Wreserved-user-defined-literal.C View 1 chunk +29 lines, -0 lines 0 comments Download
M libcpp/include/cpplib.h View 2 chunks +6 lines, -1 line 0 comments Download
M libcpp/init.c View 1 chunk +1 line, -0 lines 0 comments Download
M libcpp/lex.c View 2 chunks +38 lines, -6 lines 0 comments Download

Messages

Total messages: 12
Ollie Wild
Add new option, -Wreserved-user-defined-literal. This option, which is enabled by default, causes the preprocessor to ...
9 years, 5 months ago (2012-04-23 02:54:02 UTC) #1
Jeffrey Yasskin (google)
Could you try to get this into mainline instead of just the google branches? In ...
9 years, 5 months ago (2012-04-23 03:11:18 UTC) #2
Ollie Wild
I'd like to get this into the google branches first because this is blocking testing. ...
9 years, 5 months ago (2012-04-23 03:14:08 UTC) #3
Jeffrey Yasskin (google)
Let's let the discussion _start_ before assuming it'll be protracted. ;) I don't think it'll ...
9 years, 5 months ago (2012-04-23 03:29:52 UTC) #4
Ollie Wild
Okay, I'll send out a trunk patch for review now, too. Ollie On Sun, Apr ...
9 years, 5 months ago (2012-04-23 03:32:13 UTC) #5
Jeffrey Yasskin (google)
Thanks! On Sun, Apr 22, 2012 at 8:32 PM, Ollie Wild <aaw@google.com> wrote: > Okay, ...
9 years, 5 months ago (2012-04-23 03:36:11 UTC) #6
jwakely
On 23 April 2012 04:10, Jeffrey Yasskin wrote: > Could you try to get this ...
9 years, 5 months ago (2012-04-23 08:11:35 UTC) #7
Diego Novillo
On Sun, Apr 22, 2012 at 22:54, Ollie Wild <aaw@google.com> wrote: > Add new option, ...
9 years, 5 months ago (2012-04-23 19:39:53 UTC) #8
Ollie Wild
On Mon, Apr 23, 2012 at 2:39 PM, Diego Novillo <dnovillo@google.com> wrote: > > This ...
9 years, 5 months ago (2012-04-23 19:53:25 UTC) #9
Diego Novillo
On 4/23/12 3:53 PM, Ollie Wild wrote: > On Mon, Apr 23, 2012 at 2:39 ...
9 years, 5 months ago (2012-04-23 20:51:31 UTC) #10
Ollie Wild
On Mon, Apr 23, 2012 at 3:51 PM, Diego Novillo <dnovillo@google.com> wrote: > > Great, ...
9 years, 5 months ago (2012-04-23 22:16:42 UTC) #11
Diego Novillo
9 years, 5 months ago (2012-04-24 13:12:30 UTC) #12
On Mon, Apr 23, 2012 at 18:16, Ollie Wild <aaw@google.com> wrote:

> If the trunk change isn't approved today, I'll probably go ahead and
> check in *that* patch to google/integration.  The main difference
> relative to this is that the option name has been changed to
> -Wliteral-suffix.  *That* part has been approved, so hopefully any
> additional adjustments will be minor.

Sounds good.  Thanks.


Diego.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b