diff mbox

Get rid of assembly file with -fdump-ada-spec

Message ID 2209863.AodXsCb8Dm@polaris
State New
Headers show

Commit Message

Eric Botcazou June 4, 2015, 3:23 p.m. UTC 
Hi,

currently GCC generates an assembly file when run on C/C++ header files with 
the -fdump-ada-spec switch.  This is unexpected since -fdump-ada-spec is used 
only to output an Ada binding.

This patch fixes the specs for C/C++ header files in order to pass "-o %g.s"  
to cc1/cc1plus in this case, so that assembly files are removed at exit.  It 
also fixes the processing of the -fsyntax-only flag for C header file so that 
the "-o" argument is passed only once, as the spec for C++ header files does. 

Tested on x86_64-suse-linux, OK for the mainline?


2015-06-04  Pierre-Marie de Rodat  <derodat@adacore.com>

	* gcc.c (default_compilers): Pass "-o %g.s" to cc1 for headers even if
	-fdump-ada-spec is passed but not if -fsyntax-only is.
cp/
	* lang-specs.h: Likewise.
diff mbox

Patch

Index: gcc.c
===================================================================
--- gcc.c	(revision 224054)
+++ gcc.c	(working copy)
@@ -1176,12 +1176,14 @@  static const struct compiler default_com
 		%(cpp_options) -o %{save-temps*:%b.i} %{!save-temps*:%g.i} \n\
 		    cc1 -fpreprocessed %{save-temps*:%b.i} %{!save-temps*:%g.i} \
 			%(cc1_options)\
-                        %{!fdump-ada-spec*:-o %g.s %{!o*:--output-pch=%i.gch}\
-                        %W{o*:--output-pch=%*}}%V}\
+			%{!fsyntax-only:-o %g.s \
+			    %{!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)\
-                    %{!fdump-ada-spec*:-o %g.s %{!o*:--output-pch=%i.gch}\
-                    %W{o*:--output-pch=%*}}%V}}}}}}", 0, 0, 0},
+		    %{!fsyntax-only:-o %g.s \
+		        %{!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},
Index: cp/lang-specs.h
===================================================================
--- cp/lang-specs.h	(revision 224054)
+++ cp/lang-specs.h	(working copy)
@@ -47,8 +47,9 @@  along with GCC; see the file COPYING3.
       cc1plus %{save-temps*|no-integrated-cpp:-fpreprocessed %{save-temps*:%b.ii} %{!save-temps*:%g.ii}}\
 	      %{!save-temps*:%{!no-integrated-cpp:%(cpp_unique_options)}}\
 	%(cc1_options) %2\
-	%{!fsyntax-only:%{!fdump-ada-spec*:-o %g.s %{!o*:--output-pch=%i.gch}\
-        %W{o*:--output-pch=%*}}%V}}}}",
+	%{!fsyntax-only:-o %g.s \
+	    %{!fdump-ada-spec*:%{!o*:--output-pch=%i.gch}\
+			       %W{o*:--output-pch=%*}}%V}}}}",
      CPLUSPLUS_CPP_SPEC, 0, 0},
   {"@c++",
     "%{E|M|MM:cc1plus -E %(cpp_options) %2 %(cpp_debug_options)}\