OpenE2K/gcc
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

c++: Fix langspecs with -fsyntax-only [PR98591]

Browse Source 
-fsyntax-only is handled specially in the driver and causes it to add
 '-o /dev/null' (or a suitable OS-specific variant thereof).  PCH is
 handled in the language driver.  I'd not sufficiently protected the
 -fmodule-only action of adding a dummy assembler from the actions of
 -fsyntax-only, so we ended up with two -o options.

	PR c++/98591
	gcc/cp/
	* lang-specs.h: Fix handling of -fmodule-only with -fsyntax-only.
This commit is contained in:
Nathan Sidwell 2021-01-15 08:48:54 -08:00
parent 5a783f42d7
commit 492b90f33d
2 changed files with 28 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
gcc/cp/lang-specs.h
View File

@ -52,9 +52,11 @@ along with GCC; see the file COPYING3. If not see
" %{!save-temps*:%{!no-integrated-cpp:%(cpp_unique_options)}}"
" %{fmodules-ts:-fmodule-header %{fpreprocessed:-fdirectives-only}}"
" %(cc1_options) %2"
" %{!fsyntax-only:"
" %{!S:-o %g.s%V}"
" %{!fsyntax-only:%{!fmodule-*:%{!fmodules-*:%{!fdump-ada-spec*:"
" %{!o*:--output-pch=%i.gch}%W{o*:--output-pch=%*}}}}}}}}",
" %{!fmodule-*:%{!fmodules-*:%{!fdump-ada-spec*:"
" %{!o*:--output-pch=%i.gch}%W{o*:--output-pch=%*}}}}}"
"}}}",
CPLUSPLUS_CPP_SPEC, 0, 0},
{"@c++-system-header",
"%{E|M|MM:cc1plus -E"
@ -68,11 +70,14 @@ along with GCC; see the file COPYING3. If not see
" %{fmodules-ts:-fdirectives-only}"
" %{save-temps*:%b.ii} %{!save-temps*:%g.ii}}"
" %{!save-temps*:%{!no-integrated-cpp:%(cpp_unique_options)}}"
" %{fmodules-ts:-fmodule-header=system %{fpreprocessed:-fdirectives-only}}"
" %{fmodules-ts:-fmodule-header=system"
" %{fpreprocessed:-fdirectives-only}}"
" %(cc1_options) %2"
" %{!fsyntax-only:"
" %{!S:-o %g.s%V}"
" %{!fsyntax-only:%{!fmodule-*:%{!fmodules-*:%{!fdump-ada-spec*:"
" %{!o*:--output-pch=%i.gch}%W{o*:--output-pch=%*}}}}}}}}",
" %{!fmodule-*:%{!fmodules-*:%{!fdump-ada-spec*:"
" %{!o*:--output-pch=%i.gch}%W{o*:--output-pch=%*}}}}}"
"}}}",
CPLUSPLUS_CPP_SPEC, 0, 0},
{"@c++-user-header",
"%{E|M|MM:cc1plus -E"
@ -88,9 +93,11 @@ along with GCC; see the file COPYING3. If not see
" %{!save-temps*:%{!no-integrated-cpp:%(cpp_unique_options)}}"
" %{fmodules-ts:-fmodule-header=user %{fpreprocessed:-fdirectives-only}}"
" %(cc1_options) %2"
" %{!fsyntax-only:"
" %{!S:-o %g.s%V}"
" %{!fsyntax-only:%{!fmodule-*:%{!fmodules-*:%{!fdump-ada-spec*:"
" %{!o*:--output-pch=%i.gch}%W{o*:--output-pch=%*}}}}}}}}",
" %{!fmodule-*:%{!fmodules-*:%{!fdump-ada-spec*:"
" %{!o*:--output-pch=%i.gch}%W{o*:--output-pch=%*}}}}}"
"}}}",
CPLUSPLUS_CPP_SPEC, 0, 0},
{"@c++",
"%{E|M|MM:cc1plus -E %(cpp_options) %2 %(cpp_debug_options)}"
@ -101,13 +108,16 @@ along with GCC; see the file COPYING3. If not see
" %{save-temps*:%b.ii} %{!save-temps*:%g.ii}}"
" %{!save-temps*:%{!no-integrated-cpp:%(cpp_unique_options)}}"
" %(cc1_options) %2"
" %{!fsyntax-only:"
" %{fmodule-only:%{!S:-o %g.s%V}}"
" %{!fsyntax-only:%{!fmodule-only:%(invoke_as)}}}}}",
" %{!fmodule-only:%(invoke_as)}}"
"}}}",
CPLUSPLUS_CPP_SPEC, 0, 0},
{".ii", "@c++-cpp-output", 0, 0, 0},
{"@c++-cpp-output",
"%{!E:%{!M:%{!MM:"
" cc1plus -fpreprocessed %i %(cc1_options) %2"
" %{!fsyntax-only:"
" %{fmodule-only:%{!S:-o %g.s%V}}"
" %{!fsyntax-only:%{!fmodule-only:%{!fmodule-header*:"
" %(invoke_as)}}}}}}", 0, 0, 0},
" %{!fmodule-only:%{!fmodule-header*:%(invoke_as)}}}"
"}}}", 0, 0, 0},

3
gcc/testsuite/g++.dg/modules/pr98591.H Normal file
View File

@ -0,0 +1,3 @@
// { dg-additional-options {-fmodules-ts -fmodule-header -fsyntax-only} }
// PR 98591 -fsyntax-only -> output filename specified twice
// specs are hard