Re: RFA [reload]: Fix PR other/58545

2013-10-30 Thread Jeff Law
On 10/03/13 14:23, Joern Rennecke wrote: 2013-10-02 Joern Rennecke PR other/58545 * reload1.c (update_eliminables_and_spill): New function, broken out of reload. (reload): Use it. Check for frame size change after frame size alignment, and call update_e

Re: RFA [reload]: Fix PR other/58545

2013-10-30 Thread Jeff Law
On 10/11/13 08:40, Joern Rennecke wrote: On 11 October 2013 04:53, Jeff Law 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, tha

Re: RFA [reload]: Fix PR other/58545

2013-10-11 Thread Joern Rennecke
On 11 October 2013 04:53, Jeff Law 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 us

Re: RFA [reload]: Fix PR other/58545

2013-10-10 Thread Jeff Law
On 10/03/13 14:23, Joern Rennecke wrote: To make sure that reloading is done with up-to-date register eliminations, I call update_eliminables also in the code path that continues for a frame size change. Not directly, since I need the code to spill the no longer eliminated registers; I've factor