Re: [RFA] fetch result of locdesc expressions as integer (not address)

Ulrich Weigand
2011-10-04T23:06:12Z

Joel Brobecker wrote:
> > It seems the problem is a bit more complex: different callers
> > of decode_locdesc have different expectations. As the comment
> > before the routine says:
> >
> > NOTE drow/2003-11-18: This function is called in two situations
> > now: for the address of static or global variables (partial symbols
> > only) and for offsets into structures which are expected to be
> > (more or less) constant.
>
> That's true, but my interpretation was the callers of decode_locdesc
> should know which they expect it to me, and thus know to apply
> the integer_to_address correction.

Well, maybe they should, but right now they don't, and neither does
your patch add any such correction. The point I was trying to make
is that therefore, your patch as it is, while fixing one class of
bugs on some targets, may simultaneously introduce a different class
of bugs on other targets. I'm not sure this is a good idea ...

Bye,
Ulrich

--
Dr. Ulrich Weigand
GNU Toolchain for Linux on System z and Cell BE
Ulrich.Weigand@de.ibm.com