This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: 3.3.1/3.4 PATCH: Don't pass --gstabs to Solaris 2/Intel native as


Jim Wilson writes:

> Rainer Orth wrote:
> > 	* configure.in (gcc_cv_as_gstabs_flag): Disable if assembler warns.
> > 	* configure: Regenerate.
> > 	Fixes PR driver/9362.
> 
> It seems reasonable, except that you are always running the assembler 
> twice now, even if the first test fails.
> 
> How about putting the second if inside the first one?  I think it is OK 
> with that change.  This is done for the gcc_cv_as_leb128 test for instance.

ok, this is what I've checked in (after re-running configure to make sure I
haven't introduced any syntax errors).

	Rainer

-----------------------------------------------------------------------------
Rainer Orth, Faculty of Technology, Bielefeld University


Index: gcc/configure.in
===================================================================
RCS file: /cvs/gcc/gcc/gcc/configure.in,v
retrieving revision 1.689
diff -u -p -r1.689 configure.in
--- gcc/configure.in	17 Jun 2003 21:53:45 -0000	1.689
+++ gcc/configure.in	20 Jun 2003 13:45:40 -0000
@@ -2560,6 +2560,13 @@ elif test x$gcc_cv_as != x -a x"$insn" !
 	# ??? This fails with non-gnu grep.
 	if $gcc_cv_as --gstabs -o conftest.o conftest.s > /dev/null 2>&1 ; then
 	  gcc_cv_as_gstabs_flag="yes"
+	  # The native Solaris 9/Intel assembler doesn't understand --gstabs
+	  # and warns about it, but still exits successfully.  So check for
+	  # this.
+	  if $gcc_cv_as --gstabs -o conftest.o conftest.s 2>&1 | \
+		grep -i warning > /dev/null ; then
+	    gcc_cv_as_gstabs_flag="no"
+	  fi
 	fi
 	rm -f conftest.s conftest.o
 fi


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]