From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1851) id D65C5385AC22; Mon, 9 May 2022 08:57:13 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org D65C5385AC22 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Martin Liska To: gcc-cvs@gcc.gnu.org Subject: [gcc(refs/users/marxin/heads/PR105355-joined-separate-opts)] opts: do not allow Separate+Joined ending with = X-Act-Checkin: gcc X-Git-Author: Martin Liska X-Git-Refname: refs/users/marxin/heads/PR105355-joined-separate-opts X-Git-Oldrev: 22d9c8802add09a93308319fc37dd3a0f1125393 X-Git-Newrev: e76e6dbd2b235aef080b9782a9a144cb906fe478 Message-Id: <20220509085713.D65C5385AC22@sourceware.org> Date: Mon, 9 May 2022 08:57:13 +0000 (GMT) X-BeenThere: gcc-cvs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 09 May 2022 08:57:13 -0000 https://gcc.gnu.org/g:e76e6dbd2b235aef080b9782a9a144cb906fe478 commit e76e6dbd2b235aef080b9782a9a144cb906fe478 Author: Martin Liska Date: Tue Apr 26 11:41:09 2022 +0200 opts: do not allow Separate+Joined ending with = Do not allow strange option format: -msmall-data-limit= 11. For -output-pch we need to use Separate syntax as lang spec rules automatically append a space. PR target/105355 gcc/ChangeLog: * config/riscv/riscv.opt: Remove Separate from -msmall-data-limit=. * optc-gen.awk: Report error for the described situation. * gcc.cc: Use Separate syntax. * opts.cc (gen_command_line_string): Change option name. gcc/c-family/ChangeLog: * c-opts.cc (c_common_handle_option): Change option name. * c.opt: Remove Joined and use Separate option. gcc/cp/ChangeLog: * lang-specs.h: Use Separate syntax. gcc/objc/ChangeLog: * lang-specs.h: Use Separate syntax. gcc/objcp/ChangeLog: * lang-specs.h: Use Separate syntax. Diff: --- gcc/c-family/c-opts.cc | 2 +- gcc/c-family/c.opt | 4 ++-- gcc/config/riscv/riscv.opt | 2 +- gcc/cp/lang-specs.h | 6 +++--- gcc/gcc.cc | 8 ++++---- gcc/objc/lang-specs.h | 8 ++++---- gcc/objcp/lang-specs.h | 2 +- gcc/optc-gen.awk | 7 +++++++ gcc/opts.cc | 2 +- 9 files changed, 24 insertions(+), 17 deletions(-) diff --git a/gcc/c-family/c-opts.cc b/gcc/c-family/c-opts.cc index a341a061758..da377f09bba 100644 --- a/gcc/c-family/c-opts.cc +++ b/gcc/c-family/c-opts.cc @@ -303,7 +303,7 @@ c_common_handle_option (size_t scode, const char *arg, HOST_WIDE_INT value, result = false; break; - case OPT__output_pch_: + case OPT__output_pch: pch_file = arg; break; diff --git a/gcc/c-family/c.opt b/gcc/c-family/c.opt index 07da40ef43b..035b1de0d84 100644 --- a/gcc/c-family/c.opt +++ b/gcc/c-family/c.opt @@ -135,8 +135,8 @@ C ObjC C++ ObjC++ Separate Alias(o) -output= C ObjC C++ ObjC++ Joined Alias(o) --output-pch= -C ObjC C++ ObjC++ Joined Separate +-output-pch +C ObjC C++ ObjC++ Separate -pedantic C ObjC C++ ObjC++ Alias(pedantic) diff --git a/gcc/config/riscv/riscv.opt b/gcc/config/riscv/riscv.opt index 492aad12324..84c8cf5a2de 100644 --- a/gcc/config/riscv/riscv.opt +++ b/gcc/config/riscv/riscv.opt @@ -92,7 +92,7 @@ Target RejectNegative Joined Var(riscv_cpu_string) -mcpu=PROCESSOR Use architecture of and optimize the output for PROCESSOR. msmall-data-limit= -Target Joined Separate UInteger Var(g_switch_value) Init(8) +Target Joined UInteger Var(g_switch_value) Init(8) -msmall-data-limit=N Put global and static data smaller than bytes into a special section (on some targets). msave-restore diff --git a/gcc/cp/lang-specs.h b/gcc/cp/lang-specs.h index f35c9fab76b..dbda5a7b2e4 100644 --- a/gcc/cp/lang-specs.h +++ b/gcc/cp/lang-specs.h @@ -55,7 +55,7 @@ along with GCC; see the file COPYING3. If not see " %{!fsyntax-only:" " %{!S:-o %g.s%V}" " %{!fmodule-*:%{!fmodules-*:%{!fdump-ada-spec*:" - " %{!o*:--output-pch=%i.gch}%W{o*:--output-pch=%*}}}}}" + " %{!o*:--output-pch %i.gch}%W{o*:--output-pch %*}}}}}" "}}}", CPLUSPLUS_CPP_SPEC, 0, 0}, {"@c++-system-header", @@ -76,7 +76,7 @@ along with GCC; see the file COPYING3. If not see " %{!fsyntax-only:" " %{!S:-o %g.s%V}" " %{!fmodule-*:%{!fmodules-*:%{!fdump-ada-spec*:" - " %{!o*:--output-pch=%i.gch}%W{o*:--output-pch=%*}}}}}" + " %{!o*:--output-pch %i.gch}%W{o*:--output-pch %*}}}}}" "}}}", CPLUSPLUS_CPP_SPEC, 0, 0}, {"@c++-user-header", @@ -96,7 +96,7 @@ along with GCC; see the file COPYING3. If not see " %{!fsyntax-only:" " %{!S:-o %g.s%V}" " %{!fmodule-*:%{!fmodules-*:%{!fdump-ada-spec*:" - " %{!o*:--output-pch=%i.gch}%W{o*:--output-pch=%*}}}}}" + " %{!o*:--output-pch %i.gch}%W{o*:--output-pch %*}}}}}" "}}}", CPLUSPLUS_CPP_SPEC, 0, 0}, {"@c++", diff --git a/gcc/gcc.cc b/gcc/gcc.cc index bb07cc244e3..299e09c4f54 100644 --- a/gcc/gcc.cc +++ b/gcc/gcc.cc @@ -1462,13 +1462,13 @@ static const struct compiler default_compilers[] = cc1 -fpreprocessed %{save-temps*:%b.i} %{!save-temps*:%g.i} \ %(cc1_options)\ %{!fsyntax-only:%{!S:-o %g.s} \ - %{!fdump-ada-spec*:%{!o*:--output-pch=%i.gch}\ - %W{o*:--output-pch=%*}}%V}}\ + %{!fdump-ada-spec*:%{!o*:--output-pch %i.gch}\ + %W{o*:--output-pch %*}}%V}}\ %{!save-temps*:%{!traditional-cpp:%{!no-integrated-cpp:\ cc1 %(cpp_unique_options) %(cc1_options)\ %{!fsyntax-only:%{!S:-o %g.s} \ - %{!fdump-ada-spec*:%{!o*:--output-pch=%i.gch}\ - %W{o*:--output-pch=%*}}%V}}}}}}}", 0, 0, 0}, + %{!fdump-ada-spec*:%{!o*:--output-pch %i.gch}\ + %W{o*:--output-pch %*}}%V}}}}}}}", 0, 0, 0}, {".i", "@cpp-output", 0, 0, 0}, {"@cpp-output", "%{!M:%{!MM:%{!E:cc1 -fpreprocessed %i %(cc1_options) %{!fsyntax-only:%(invoke_as)}}}}", 0, 0, 0}, diff --git a/gcc/objc/lang-specs.h b/gcc/objc/lang-specs.h index 1a785ac3a88..049166cb114 100644 --- a/gcc/objc/lang-specs.h +++ b/gcc/objc/lang-specs.h @@ -41,12 +41,12 @@ along with GCC; see the file COPYING3. If not see %eGNU Objective C no longer supports traditional compilation}\ %{save-temps*|no-integrated-cpp:cc1obj -E %(cpp_options) -o %{save-temps*:%b.mi} %{!save-temps*:%g.mi} \n\ cc1obj -fpreprocessed %b.mi %(cc1_options) %{print-objc-runtime-info} %{gen-decls}\ - -o %g.s %{!o*:--output-pch=%i.gch}\ - %W{o*:--output-pch=%*}%V}\ + -o %g.s %{!o*:--output-pch %i.gch}\ + %W{o*:--output-pch %*}%V}\ %{!save-temps*:%{!no-integrated-cpp:\ cc1obj %(cpp_unique_options) %(cc1_options) %{print-objc-runtime-info} %{gen-decls}\ - -o %g.s %{!o*:--output-pch=%i.gch}\ - %W{o*:--output-pch=%*}%V}}}}}", 0, 0, 0}, + -o %g.s %{!o*:--output-pch %i.gch}\ + %W{o*:--output-pch %*}%V}}}}}", 0, 0, 0}, {".mi", "@objective-c-cpp-output", 0, 0, 0}, {"@objective-c-cpp-output", "%{!M:%{!MM:%{!E:cc1obj -fpreprocessed %i %(cc1_options) %{print-objc-runtime-info} %{gen-decls}\ diff --git a/gcc/objcp/lang-specs.h b/gcc/objcp/lang-specs.h index 2d70f3090f7..49f78a7cdfd 100644 --- a/gcc/objcp/lang-specs.h +++ b/gcc/objcp/lang-specs.h @@ -37,7 +37,7 @@ along with GCC; see the file COPYING3. If not see cc1objplus %{save-temps*|no-integrated-cpp:-fpreprocessed %{save-temps*:%b.mii} %{!save-temps*:%g.mii}}\ %{!save-temps*:%{!no-integrated-cpp:%(cpp_unique_options)}}\ %(cc1_options) %2\ - -o %g.s %{!o*:--output-pch=%i.gch} %W{o*:--output-pch=%*}%V}}}", + -o %g.s %{!o*:--output-pch %i.gch} %W{o*:--output-pch %*}%V}}}", CPLUSPLUS_CPP_SPEC, 0, 0}, {"@objective-c++", "%{E|M|MM:cc1objplus -E %(cpp_options) %2 %(cpp_debug_options)}\ diff --git a/gcc/optc-gen.awk b/gcc/optc-gen.awk index b51688d284f..595d3cff677 100644 --- a/gcc/optc-gen.awk +++ b/gcc/optc-gen.awk @@ -342,6 +342,13 @@ for (i = 0; i < n_opts; i++) { len = length (opts[i]); enum = opt_enum(opts[i]) + # Do not allow Joined and Separate properties if + # an options ends with '='. + if (flag_set_p("Joined", flags[i]) && flag_set_p("Separate", flags[i]) && opts[i] ~ "=$") { + print "#error Option '" opts[i] "' ending with '=' cannot have " \ + "both Joined and Separate properties" + } + # If this switch takes joined arguments, back-chain all # subsequent switches to it for which it is a prefix. If # a later switch S is a longer prefix of a switch T, T diff --git a/gcc/opts.cc b/gcc/opts.cc index 2ffbf429b7b..f9064d8680e 100644 --- a/gcc/opts.cc +++ b/gcc/opts.cc @@ -3611,7 +3611,7 @@ gen_command_line_string (cl_decoded_option *options, case OPT_SPECIAL_input_file: case OPT_grecord_gcc_switches: case OPT_frecord_gcc_switches: - case OPT__output_pch_: + case OPT__output_pch: case OPT_fdiagnostics_show_location_: case OPT_fdiagnostics_show_option: case OPT_fdiagnostics_show_caret: