This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: RFA: general prologue analysis framework


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]