Re: Patch to fix gcc.c-torture/compile/20010102-1.c on IA64 HP-UX

[Jeff Law <law@redhat.com>]

Peter Bergner wrote:
> On Thu, 2008-10-30 at 10:52 -0700, Steve Ellcey wrote:
>   
>> I don't want to just remove my patch but it might make sense to
>> change regrename to only set the attributes (including pointer) for
>> non-hard registers if that fixes your problem and doesn't cause a
>> regression for me.
>>     
>
> I actually have already asked Luis to test that.  My guess is that
> is what we eventually will end up wanting to go with, but you might
> then need to use the patch I posted for ia64/predicates.md, namely:
>
> Index: config/ia64/predicates.md
> ===================================================================
> --- config/ia64/predicates.md   (revision 140417)
> +++ config/ia64/predicates.md   (working copy)
> @@ -585,6 +585,6 @@ (define_predicate "ar_pfs_reg_operand"
>  (define_predicate "basereg_operand"
>    (match_operand 0 "register_operand")
>  {
> -  return REG_P (op) && REG_POINTER (op);
> +  return REG_P (op) && REG_POINTER (regno_reg_rtx[ORIGINAL_REGNO (op)]);
>  })
>
> If you recall, that did solve the ICE you were hitting.  The rerename.c
> change was just handling a case that looked like should be handled, but
> wasn't necessary for fixing your IC
>   
We shouldn't be hacking up backends to deal with this problem, it's 
entirely the wrong approach.


jeff