c.opt (Wcomment): Use CPP, Var and LangEnabledBy.

gcc/c-family/ChangeLog:

2014-08-22  Manuel López-Ibáñez  <manu@gcc.gnu.org>

	* c.opt (Wcomment): Use CPP, Var and LangEnabledBy.
	(Wmultichar): Likewise.
	(Wdate-time): Use C-family languages instead of Common. Use CPP
	and Var.
	* c-opts.c (c_common_handle_option): Do not handle the above
	options here.
	(sanitize_cpp_opts): Likewise.

gcc/testsuite/ChangeLog:

2014-08-22  Manuel López-Ibáñez  <manu@gcc.gnu.org>

	* g++.dg/warn/wdate-time.C: Remove.
	* gcc.dg/wdate-time.c: Move from here...
	* c-c++-common/wdate-time.c: ... to here.

From-SVN: r214350
parent 1d455520
2014-08-22 Manuel López-Ibáñez <manu@gcc.gnu.org>
* c.opt (Wcomment): Use CPP, Var and LangEnabledBy.
(Wmultichar): Likewise.
(Wdate-time): Use C-family languages instead of Common. Use CPP
and Var.
* c-opts.c (c_common_handle_option): Do not handle the above
options here.
(sanitize_cpp_opts): Likewise.
2014-08-21 Manuel López-Ibáñez <manu@gcc.gnu.org> 2014-08-21 Manuel López-Ibáñez <manu@gcc.gnu.org>
PR fortran/44054 PR fortran/44054
......
...@@ -382,7 +382,6 @@ c_common_handle_option (size_t scode, const char *arg, int value, ...@@ -382,7 +382,6 @@ c_common_handle_option (size_t scode, const char *arg, int value,
/* ??? Don't add new options here. Use LangEnabledBy in c.opt. */ /* ??? Don't add new options here. Use LangEnabledBy in c.opt. */
cpp_opts->warn_trigraphs = value; cpp_opts->warn_trigraphs = value;
cpp_opts->warn_comments = value;
cpp_opts->warn_num_sign_change = value; cpp_opts->warn_num_sign_change = value;
break; break;
...@@ -390,10 +389,6 @@ c_common_handle_option (size_t scode, const char *arg, int value, ...@@ -390,10 +389,6 @@ c_common_handle_option (size_t scode, const char *arg, int value,
cpp_opts->warn_builtin_macro_redefined = value; cpp_opts->warn_builtin_macro_redefined = value;
break; break;
case OPT_Wcomment:
cpp_opts->warn_comments = value;
break;
case OPT_Wc___compat: case OPT_Wc___compat:
cpp_opts->warn_cxx_operator_names = value; cpp_opts->warn_cxx_operator_names = value;
break; break;
...@@ -421,12 +416,8 @@ c_common_handle_option (size_t scode, const char *arg, int value, ...@@ -421,12 +416,8 @@ c_common_handle_option (size_t scode, const char *arg, int value,
case OPT_Wmissing_include_dirs: case OPT_Wmissing_include_dirs:
cpp_opts->warn_missing_include_dirs = value; cpp_opts->warn_missing_include_dirs = value;
break; break;
case OPT_Wmultichar:
cpp_opts->warn_multichar = value;
break;
case OPT_Wnormalized_: case OPT_Wnormalized_:
/* FIXME: Move all this to c.opt. */
if (kind == DK_ERROR) if (kind == DK_ERROR)
{ {
gcc_assert (!arg); gcc_assert (!arg);
...@@ -1310,7 +1301,6 @@ sanitize_cpp_opts (void) ...@@ -1310,7 +1301,6 @@ sanitize_cpp_opts (void)
cpp_opts->unsigned_char = !flag_signed_char; cpp_opts->unsigned_char = !flag_signed_char;
cpp_opts->stdc_0_in_system_headers = STDC_0_IN_SYSTEM_HEADERS; cpp_opts->stdc_0_in_system_headers = STDC_0_IN_SYSTEM_HEADERS;
cpp_opts->warn_date_time = cpp_warn_date_time;
cpp_opts->cpp_warn_c90_c99_compat = warn_c90_c99_compat; cpp_opts->cpp_warn_c90_c99_compat = warn_c90_c99_compat;
/* Wlong-long is disabled by default. It is enabled by: /* Wlong-long is disabled by default. It is enabled by:
......
...@@ -328,7 +328,7 @@ C ObjC C++ ObjC++ Var(warn_clobbered) Warning EnabledBy(Wextra) ...@@ -328,7 +328,7 @@ C ObjC C++ ObjC++ Var(warn_clobbered) Warning EnabledBy(Wextra)
Warn about variables that might be changed by \"longjmp\" or \"vfork\" Warn about variables that might be changed by \"longjmp\" or \"vfork\"
Wcomment Wcomment
C ObjC C++ ObjC++ Warning C ObjC C++ ObjC++ CPP(warn_comments) Var(cpp_warn_comment) Warning LangEnabledBy(C ObjC C++ ObjC++,Wall)
Warn about possibly nested block comments, and C++ comments spanning more than one physical line Warn about possibly nested block comments, and C++ comments spanning more than one physical line
Wcomments Wcomments
...@@ -598,7 +598,7 @@ Wmudflap ...@@ -598,7 +598,7 @@ Wmudflap
C ObjC C++ ObjC++ Ignore Warn(switch %qs is no longer supported) C ObjC C++ ObjC++ Ignore Warn(switch %qs is no longer supported)
Wmultichar Wmultichar
C ObjC C++ ObjC++ Warning C ObjC C++ ObjC++ CPP(warn_multichar) Var(cpp_warn_multichar) Warning
Warn about use of multi-character character constants Warn about use of multi-character character constants
Wnarrowing Wnarrowing
...@@ -702,7 +702,7 @@ C ObjC C++ ObjC++ Var(warn_pragmas) Init(1) Warning ...@@ -702,7 +702,7 @@ C ObjC C++ ObjC++ Var(warn_pragmas) Init(1) Warning
Warn about misuses of pragmas Warn about misuses of pragmas
Wdate-time Wdate-time
Common Var(cpp_warn_date_time) Warning C ObjC C++ ObjC++ CPP(warn_date_time) Var(cpp_warn_date_time) Warning
Warn about __TIME__, __DATE__ and __TIMESTAMP__ usage Warn about __TIME__, __DATE__ and __TIMESTAMP__ usage
Wproperty-assign-default Wproperty-assign-default
......
2014-08-22 Manuel López-Ibáñez <manu@gcc.gnu.org>
* g++.dg/warn/wdate-time.C: Remove.
* gcc.dg/wdate-time.c: Move from here...
* c-c++-common/wdate-time.c: ... to here.
2014-08-22 Joost VandeVondele <Joost.VandeVondele@mat.ethz.ch> 2014-08-22 Joost VandeVondele <Joost.VandeVondele@mat.ethz.ch>
* gfortran.dg/use_without_only_1.f90: New test. * gfortran.dg/use_without_only_1.f90: New test.
......
/* { dg-do compile } */
/* { dg-options "-Wdate-time" } */
const char time[] = __TIME__; /* { dg-warning "might prevent reproducible builds" } */
const char date[] = __DATE__; /* { dg-warning "might prevent reproducible builds" } */
const char timestamp[] = __TIMESTAMP__; /* { dg-warning "might prevent reproducible builds" } */
/* { dg-do compile } */
/* { dg-options "-Wdate-time" } */
const char time[] = __TIME__; /* { dg-warning "might prevent reproducible builds" } */
const char date[] = __DATE__; /* { dg-warning "might prevent reproducible builds" } */
const char timestamp[] = __TIMESTAMP__; /* { dg-warning "might prevent reproducible builds" } */
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment