Patch for PR28490 (IA64 symbol+offset)

Steve Ellcey sje@cup.hp.com
Fri Sep 15 22:21:00 GMT 2006


> On Mon, 2006-09-11 at 10:08 -0700, Steve Ellcey wrote:
> > I tested this patch over the weekend and had no regressions, do you want
> > to check it in?
> 
> I've been a bit overwhelmed here, and have gotten well behind on my gcc
> work.  I'm OK with checking in my patch as a short term solution, though
> I don't like to checking in patches I haven't tested.  I don't want to
> accidentally check in a typo.  But it is the weekend now, almost, so I
> can do a quick perfunctory build and check it in.
> -- 
> Jim Wilson, GNU Tools Support, http://www.specifix.com

I actually found I had to make one tweak to it, it worked fine on Linux
but when I tested HP-UX I had a problem with functions.  In
ia64_legitimate_constant_p I changed:

         if (any_offset_symbol_operand (op, GET_MODE (op)))
           return true;

to:

         if (any_offset_symbol_operand (op, GET_MODE (op))
             || function_operand (op, GET_MODE (op)))
           return true;

I am not sure why this only seemed to affect HP-UX but adding the
check for function_operand seemed to fix my problems.

Steve Ellcey
sje@cup.hp.com



More information about the Gcc-patches mailing list