[PATCH 3.0] leb128 test yields 'Linux: integer expression expected'

[Phil Edwards]

On Sat, Feb 02, 2002 at 07:54:44PM -0800, Richard Henderson wrote:
> On Fri, Feb 01, 2002 at 06:27:55PM -0500, Phil Edwards wrote:
> > but should probably be tested on other GNU binutils-using platforms
> > before applying.  My Red Hat boxes are production machines with no spare
> > computer cycles.
> 
> Red Hat uses "GNU assembler 2.10.90" so this will work fine there.
> 
> The patch is ok for mainline.  IIRC we're now in code freeze for
> the 3.0 branch so you need Mark to ack the patch.

I had not looked at mainline yet, and now that I do it seems this is
already fixed there, in a different way.

Mark approved the branch patch in private email, and I have now checked
it in:


2002-02-05  Phil Edwards  <pme@gcc.gnu.org>

	* configure.in (leb128 support):  Test as versions more carefully.
	* configure:  Regenerated.


Index: configure.in
===================================================================
RCS file: /cvs/gcc/gcc/gcc/configure.in,v
retrieving revision 1.483.2.30
diff -u -3 -p -r1.483.2.30 configure.in
--- configure.in	2001/12/04 19:42:38	1.483.2.30
+++ configure.in	2002/02/05 22:08:59
@@ -1328,10 +1328,10 @@ EOF
 		# arbitrary sections are supported and try the test.
 		as_ver=`$gcc_cv_as --version 2>/dev/null | head -1`
 		if echo "$as_ver" | grep GNU > /dev/null; then
-			as_ver=`echo $as_ver | sed -e 's/.* //'`
-			as_major=`echo $as_ver | sed 's/\..*//'`
 changequote(,)dnl
-			as_minor=`echo $as_ver | sed 's/[^.]*\.\([0-9]*\).*/\1/'`
+			as_ver=`echo $as_ver | sed -e 's,[^0-9]*,,' -e 's, .*,,'`
+			as_major=`echo $as_ver | sed 's,\..*,,'`
+			as_minor=`echo $as_ver | sed 's,[^.]*\.\([0-9]*\).*,\1,'`
 changequote([,])dnl
 			if test $as_major -eq 2 -a $as_minor -lt 11; then
 				gcc_cv_as_leb128="no"