On Mon, 6 Apr 2009, Uros Bizjak wrote: > Hans-Peter Nilsson wrote: > > And again, I'd be perfectly fine with removing this weird > > LOAD_EXTEND_OP-specific "feature". I'm of half a mind to remove > > the #define from the MMIX port. > Please note, that my findings were on 4.3.4 to-be-released branch. I'm afraid > that IRA has nothing to do with this problem, IMO we are looking at plain > reload bug here.
IRA was just "needed" for MMIX to *uncover* the bug. > Removing your #define won't fix the failure, it will simply > change register allocation slightly to _hide_ the failure. I was referring to the MMIX #define LOAD_EXTEND_OP, not (the never committed) #define IRA_COVER_CLASSES. > The problem is, that reload should not inherit/substitute memory operand > (where LOAD_EXTEND_OP defines bits outside the range) with register operand > (where bits outside the range are undefined). Something like that, yes. brgds, H-P
