Re: RFA: general prologue analysis framework

Ulrich Weigand
2005-10-13T13:50:16Z

Jim Blandy wrote:

> pv_is_array_ref is strictly a utility function; there's nothing in
> there you couldn't do with the existing primitives. It was used in
> the original s390 analyzer, but nothing uses it now. S390 frames have
> (had?) distinct areas set up for saving gprs and fprs, and they're
> always full-sized. I used pv_is_array_ref to recognize stores to
> those areas. The 'pv_area' functions are new; I think they'd handle
> the problem better.

I've removed the pv_is_array_ref code from the s390 analyzer, because
recent GCC (optionally) no longer uses fixed-size register save areas
in order to reduce the amount of wasted stack space.

The analyser now assumes the top-most stack slot holding an incoming
(call-saved) register value is in fact the save area for that register.
I don't need any 'area' functions at all any more ...

Bye,
Ulrich

--
Dr. Ulrich Weigand
Linux on zSeries Development
Ulrich.Weigand@de.ibm.com