This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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 [reload]: Fix PR other/58545


On 10/11/13 08:40, Joern Rennecke wrote:
On 11 October 2013 04:53, Jeff Law <law@redhat.com> wrote:
  > With your change it seems to me that we do a single round of spilling &
caller-save setup, then align the stack, then restart.  The net result being
we align the stack a lot more often.

Yes, but AFAICT, that should not result in more space being used,
because assign_stack_local uses ASLK_RECORD_PAD, so
assign_stack_local_1 will record any space
added for alignment purposes with add_frame_space, so it can be used
for a subsequent spill of suitable size.
You're right -- I wasn't aware of Bernd's work to reuse alignment paddings for real objects. It was added a few months after I'd been poking at problems in that loop of reload. Let me look at this stuff again.

jeff


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