diff options
author | Marek Polacek <polacek@redhat.com> | 2014-08-19 06:04:38 +0000 |
---|---|---|
committer | Marek Polacek <mpolacek@gcc.gnu.org> | 2014-08-19 06:04:38 +0000 |
commit | 35aff4fb82ce0f615dc5e0ad73341a1fe77cbd12 (patch) | |
tree | 252dbb8b85e09aa8d379abb28ffc5e483fa4edb2 /gcc/c/c-parser.c | |
parent | fdec323d35ab9bf489e374c0545621bc90dd08c0 (diff) | |
download | gcc-35aff4fb82ce0f615dc5e0ad73341a1fe77cbd12.zip gcc-35aff4fb82ce0f615dc5e0ad73341a1fe77cbd12.tar.gz gcc-35aff4fb82ce0f615dc5e0ad73341a1fe77cbd12.tar.bz2 |
invoke.texi: Document -Wc99-c11-compat.
* doc/invoke.texi: Document -Wc99-c11-compat.
c-family/
* c.opt (Wc99-c11-compat): New option.
c/
* c-decl.c (diagnose_mismatched_decls): Unconditionally call
pedwarn_c99 instead of pedwarn.
(grokfield): Likewise.
(warn_defaults_to): New function.
(grokdeclarator): Call warn_defaults_to instead of pedwarn_c99.
Unconditionally call pedwarn_c99 instead of pedwarn.
(start_function): Call warn_defaults_to instead of pedwarn_c99.
(declspecs_add_scspec): Call pedwarn_c99 instead of pedwarn, don't
check flag_isoc11 before.
* c-errors.c (pedwarn_c99): Change the return type to bool.
Handle -Wc99-c11-compat.
* c-parser.c (disable_extension_diagnostics): Handle
warn_c99_c11_compat.
(restore_extension_diagnostics): Likewise.
(c_parser_static_assert_declaration_no_semi): Call pedwarn_c99
instead of pedwarn, don't check flag_isoc11 before.
(c_parser_declspecs): Likewise.
(c_parser_alignas_specifier): Likewise.
(c_parser_alignof_expression): Likewise.
(c_parser_generic_selection): Likewise.
* c-tree.h (pedwarn_c99): Update declaration.
* c-typeck.c (c_finish_return): Call pedwarn or warning_at instead
of pedwarn_c99.
testsuite/
* gcc.dg/Wc99-c11-compat-1.c: New test.
* gcc.dg/Wc99-c11-compat-2.c: New test.
* gcc.dg/Wc99-c11-compat-3.c: New test.
* gcc.dg/Wc99-c11-compat-4.c: New test.
* gcc.dg/Wc99-c11-compat-5.c: New test.
* gcc.dg/Wc99-c11-compat-6.c: New test.
* gcc.dg/Wc99-c11-compat-7.c: New test.
From-SVN: r214133
Diffstat (limited to 'gcc/c/c-parser.c')
-rw-r--r-- | gcc/c/c-parser.c | 62 |
1 files changed, 28 insertions, 34 deletions
diff --git a/gcc/c/c-parser.c b/gcc/c/c-parser.c index 6afe3eb..d588797 100644 --- a/gcc/c/c-parser.c +++ b/gcc/c/c-parser.c @@ -1076,7 +1076,11 @@ disable_extension_diagnostics (void) /* warn_c90_c99_compat has three states: -1/0/1, so we must play tricks to properly restore it. */ | ((warn_c90_c99_compat == 1) << 7) - | ((warn_c90_c99_compat == -1) << 8)); + | ((warn_c90_c99_compat == -1) << 8) + /* Similarly for warn_c99_c11_compat. */ + | ((warn_c99_c11_compat == 1) << 9) + | ((warn_c99_c11_compat == -1) << 10) + ); cpp_opts->cpp_pedantic = pedantic = 0; warn_pointer_arith = 0; cpp_opts->cpp_warn_traditional = warn_traditional = 0; @@ -1085,6 +1089,7 @@ disable_extension_diagnostics (void) warn_cxx_compat = 0; warn_overlength_strings = 0; warn_c90_c99_compat = 0; + warn_c99_c11_compat = 0; return ret; } @@ -1103,6 +1108,7 @@ restore_extension_diagnostics (int flags) warn_overlength_strings = (flags >> 6) & 1; /* See above for why is this needed. */ warn_c90_c99_compat = (flags >> 7) & 1 ? 1 : ((flags >> 8) & 1 ? -1 : 0); + warn_c99_c11_compat = (flags >> 9) & 1 ? 1 : ((flags >> 10) & 1 ? -1 : 0); } /* Possibly kinds of declarator to parse. */ @@ -2009,15 +2015,12 @@ c_parser_static_assert_declaration_no_semi (c_parser *parser) gcc_assert (c_parser_next_token_is_keyword (parser, RID_STATIC_ASSERT)); assert_loc = c_parser_peek_token (parser)->location; - if (!flag_isoc11) - { - if (flag_isoc99) - pedwarn (assert_loc, OPT_Wpedantic, + if (flag_isoc99) + pedwarn_c99 (assert_loc, OPT_Wpedantic, "ISO C99 does not support %<_Static_assert%>"); - else - pedwarn (assert_loc, OPT_Wpedantic, + else + pedwarn_c99 (assert_loc, OPT_Wpedantic, "ISO C90 does not support %<_Static_assert%>"); - } c_parser_consume_token (parser); if (!c_parser_require (parser, CPP_OPEN_PAREN, "expected %<(%>")) return; @@ -2382,15 +2385,12 @@ c_parser_declspecs (c_parser *parser, struct c_declspecs *specs, correct lvalue-to-rvalue conversions. */ if (flag_openmp) sorry ("%<_Atomic%> with OpenMP"); - if (!flag_isoc11) - { - if (flag_isoc99) - pedwarn (loc, OPT_Wpedantic, + if (flag_isoc99) + pedwarn_c99 (loc, OPT_Wpedantic, "ISO C99 does not support the %<_Atomic%> qualifier"); - else - pedwarn (loc, OPT_Wpedantic, + else + pedwarn_c99 (loc, OPT_Wpedantic, "ISO C90 does not support the %<_Atomic%> qualifier"); - } attrs_ok = true; tree value; value = c_parser_peek_token (parser)->value; @@ -3055,15 +3055,12 @@ c_parser_alignas_specifier (c_parser * parser) location_t loc = c_parser_peek_token (parser)->location; gcc_assert (c_parser_next_token_is_keyword (parser, RID_ALIGNAS)); c_parser_consume_token (parser); - if (!flag_isoc11) - { - if (flag_isoc99) - pedwarn (loc, OPT_Wpedantic, + if (flag_isoc99) + pedwarn_c99 (loc, OPT_Wpedantic, "ISO C99 does not support %<_Alignas%>"); - else - pedwarn (loc, OPT_Wpedantic, + else + pedwarn_c99 (loc, OPT_Wpedantic, "ISO C90 does not support %<_Alignas%>"); - } if (!c_parser_require (parser, CPP_OPEN_PAREN, "expected %<(%>")) return ret; if (c_parser_next_tokens_start_typename (parser, cla_prefer_id)) @@ -6579,14 +6576,14 @@ c_parser_alignof_expression (c_parser *parser) /* A diagnostic is not required for the use of this identifier in the implementation namespace; only diagnose it for the C11 spelling because of existing code using the other spellings. */ - if (!flag_isoc11 && is_c11_alignof) + if (is_c11_alignof) { if (flag_isoc99) - pedwarn (loc, OPT_Wpedantic, "ISO C99 does not support %qE", - alignof_spelling); + pedwarn_c99 (loc, OPT_Wpedantic, "ISO C99 does not support %qE", + alignof_spelling); else - pedwarn (loc, OPT_Wpedantic, "ISO C90 does not support %qE", - alignof_spelling); + pedwarn_c99 (loc, OPT_Wpedantic, "ISO C90 does not support %qE", + alignof_spelling); } c_parser_consume_token (parser); c_inhibit_evaluation_warnings++; @@ -6745,15 +6742,12 @@ c_parser_generic_selection (c_parser *parser) gcc_assert (c_parser_next_token_is_keyword (parser, RID_GENERIC)); generic_loc = c_parser_peek_token (parser)->location; c_parser_consume_token (parser); - if (!flag_isoc11) - { - if (flag_isoc99) - pedwarn (generic_loc, OPT_Wpedantic, + if (flag_isoc99) + pedwarn_c99 (generic_loc, OPT_Wpedantic, "ISO C99 does not support %<_Generic%>"); - else - pedwarn (generic_loc, OPT_Wpedantic, + else + pedwarn_c99 (generic_loc, OPT_Wpedantic, "ISO C90 does not support %<_Generic%>"); - } if (!c_parser_require (parser, CPP_OPEN_PAREN, "expected %<(%>")) return error_expr; |