Re: Another reload bug

[Richard Henderson <rth@redhat.com>]

On Thu, Apr 10, 2003 at 12:18:02AM +0200, Eric Botcazou wrote:
> Should we restore the original code as the "plan B" here?

No, I'm going to backport a small bit o code from mainline
simplify_subreg that takes care of this.

I'll commit it after testing completes; the testcase from 
this pr will be committed as g++.dg/opt/reload2.C.


r~


        Tue Jul 23 21:49:24 2002  J"orn Rennecke <joern.rennecke@superh.com>
        * simplify-rtx.c (simplify_subreg): When converting to a non-int
        mode, try to convert to an integer mode of matching size first.

Index: simplify-rtx.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/simplify-rtx.c,v
retrieving revision 1.94.2.1
diff -c -p -d -r1.94.2.1 simplify-rtx.c
*** simplify-rtx.c	6 Mar 2002 22:43:21 -0000	1.94.2.1
--- simplify-rtx.c	10 Apr 2003 20:48:36 -0000
*************** simplify_subreg (outermode, op, innermod
*** 2484,2489 ****
--- 2484,2503 ----
  	    return new;
  	}
  
+       if (GET_MODE_CLASS (outermode) != MODE_INT
+ 	  && GET_MODE_CLASS (outermode) != MODE_CC)
+ 	{
+ 	  enum machine_mode new_mode = int_mode_for_mode (outermode);
+ 
+ 	  if (new_mode != innermode || byte != 0)
+ 	    {
+ 	      op = simplify_subreg (new_mode, op, innermode, byte);
+ 	      if (! op)
+ 		return NULL_RTX;
+ 	      return simplify_subreg (outermode, op, new_mode, 0);
+ 	    }
+ 	}
+ 
        offset = byte * BITS_PER_UNIT;
        switch (GET_CODE (op))
  	{