[patch i386]: PR/41315 frame.padding0 isn't treat in all cases of i386

Jakub Jelinek jakub@redhat.com
Wed Sep 9 16:22:00 GMT 2009


On Wed, Sep 09, 2009 at 06:12:24PM +0200, Uros Bizjak wrote:
> On 09/09/2009 02:09 PM, Kai Tietz wrote:
>> Hello,
>>
>> I reviewed the prologue/epilogue handling of w64 ABI in 386 -as we
>> just had found an failure here in treating stack padding - and I found
>> that the padding0 field was inconsitant handled.
>>
>> ChangeLog
>>
>> 2009-09-09  Kai Tietz<kai.tietz@onevision.com>
>>
>> 	* config/i386.c (ix86_can_use_return_insn_p): Check for
>> 	padding0, too.
>> 	(ix86_expand_prologue): Take frame.padding0 into logic of
>> 	to_allloc checks.

Too many l's above?  Also, isn't it to_allocate?  to_alloc is never used in
i386.c.

>> 	(ix86_expand_epilogue): Likewise.
>>
>> I regression tested patches (one for 4.4, on for trunk) for
>> x86_64-pc-mingw32 and for x86_64-pc-linux64.
>> Ok for apply to head and 4.4 branch?
>>    
>
> AFAICS, frame.padding0 is cygwin/mingw thing, so you can approve it  
> yourself ;)

	Jakub



More information about the Gcc-patches mailing list