diff options
| author | Ollie Wild <aaw@google.com> | 2012-04-27 14:29:32 +0000 |
|---|---|---|
| committer | Ollie Wild <aaw@gcc.gnu.org> | 2012-04-27 14:29:32 +0000 |
| commit | 7f5f5f98c5c21e263884362a248e5891e98ac6bc (patch) | |
| tree | 4fb8c09994e5867bbbb9d8cd9a507399b19ac47e /libcpp/ChangeLog | |
| parent | 11ec770e46f664966c53b6af90a2849c1eb4dbd1 (diff) | |
| download | gcc-7f5f5f98c5c21e263884362a248e5891e98ac6bc.zip gcc-7f5f5f98c5c21e263884362a248e5891e98ac6bc.tar.gz gcc-7f5f5f98c5c21e263884362a248e5891e98ac6bc.tar.bz2 | |
Add new option, -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 which does
not begin with an underscore. According to [lex.ext]p10, this is
ill-formed.
Also modifies the preprocessor to treat such ill-formed suffixes as separate
preprocessing tokens. This is consistent with the Clang front end (see
http://llvm.org/viewvc/llvm-project?view=rev&revision=152287), and enables
backwards compatibility with code that uses formatting macros from
<inttypes.h>, as in the following code block:
int main() {
int64_t i64 = 123;
printf("My int64: %"PRId64"\n", i64);
}
Google ref b/6377711.
2012-04-27 Ollie Wild <aaw@google.com>
PR c++/52538
* gcc/c-family/c-common.c: Add CPP_W_LITERAL_SUFFIX mapping.
* gcc/c-family/c-opts.c (c_common_handle_option): Handle
OPT_Wliteral_suffix.
* gcc/c-family/c.opt: Add Wliteral-suffix.
* gcc/doc/invoke.texi (Wliteral-suffix): Document new option.
* gcc/testsuite/g++.dg/cpp0x/Wliteral-suffix.c: New test.
* libcpp/include/cpplib.h (struct cpp_options): Add new field,
warn_literal_suffix.
(CPP_W_LITERAL_SUFFIX): New enum.
* libcpp/init.c (cpp_create_reader): Default initialization of
warn_literal_suffix.
* libcpp/lex.c (lex_raw_string): Treat user-defined literals which
don't begin with '_' as separate tokens and produce a warning.
(lex_string): Ditto.
From-SVN: r186909
Diffstat (limited to 'libcpp/ChangeLog')
| -rw-r--r-- | libcpp/ChangeLog | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/libcpp/ChangeLog b/libcpp/ChangeLog index 08878b2..f7c330c 100644 --- a/libcpp/ChangeLog +++ b/libcpp/ChangeLog @@ -1,3 +1,14 @@ +2012-04-27 Ollie Wild <aaw@google.com> + + * include/cpplib.h (struct cpp_options): Add new field, + warn_literal_suffix. + (CPP_W_LITERAL_SUFFIX): New enum. + * init.c (cpp_create_reader): Default initialization of + warn_literal_suffix. + * lex.c (lex_raw_string): Treat user-defined literals which don't + begin with '_' as separate tokens and produce a warning. + (lex_string): Ditto. + 2012-04-26 Manuel López-Ibáñez <manu@gcc.gnu.org> * line-map.c (linemap_resolve_location): Synchronize comments with |