Re: [avr] fixup dwarf2 unwind info

2011-03-22 Thread Richard Henderson
On 03/22/2011 09:05 AM, Bernd Schmidt wrote: > * dwarf2out.c (dwarf2out_frame_debug_expr): Handle POST_MODIFY stores > through the stack pointer. Ok, with an update to the function block comment concerning rule 10. r~

Re: [avr] fixup dwarf2 unwind info

2011-03-22 Thread Bernd Schmidt
On 03/22/2011 05:01 PM, Richard Henderson wrote: > On 03/22/2011 05:31 AM, Bernd Schmidt wrote: >> On 03/21/2011 09:50 PM, Richard Henderson wrote: >>> Ok to commit? >> >> As far as the dwarf2out bits are concerned, yes but please change it to >> also accept POST_MODIFY - your patch is identical to

Re: [avr] fixup dwarf2 unwind info

2011-03-22 Thread Richard Henderson
On 03/22/2011 05:31 AM, Bernd Schmidt wrote: > On 03/21/2011 09:50 PM, Richard Henderson wrote: >> Ok to commit? > > As far as the dwarf2out bits are concerned, yes but please change it to > also accept POST_MODIFY - your patch is identical to some code I need > for a new target except for the rtx

Re: [avr] fixup dwarf2 unwind info

2011-03-22 Thread Richard Henderson
On 03/22/2011 05:31 AM, Bernd Schmidt wrote: > On 03/21/2011 09:50 PM, Richard Henderson wrote: >> Ok to commit? > > As far as the dwarf2out bits are concerned, yes but please change it to > also accept POST_MODIFY - your patch is identical to some code I need > for a new target except for the rtx

Re: [avr] fixup dwarf2 unwind info

2011-03-22 Thread Bernd Schmidt
On 03/21/2011 09:50 PM, Richard Henderson wrote: > Ok to commit? As far as the dwarf2out bits are concerned, yes but please change it to also accept POST_MODIFY - your patch is identical to some code I need for a new target except for the rtx code. Bernd

Re: [avr] fixup dwarf2 unwind info

2011-03-22 Thread Denis Chertykov
2011/3/21 Richard Henderson : > This is a slightly tidied patch from sometime last month, which was > buried in a thread about dwarf2 unwind info for gas.  This fixes > some totally incorrect unwind markings in the avr backend, and adds > support in the dwarf2 middle-end for handling avr's unique p

[avr] fixup dwarf2 unwind info

2011-03-21 Thread Richard Henderson
This is a slightly tidied patch from sometime last month, which was buried in a thread about dwarf2 unwind info for gas. This fixes some totally incorrect unwind markings in the avr backend, and adds support in the dwarf2 middle-end for handling avr's unique post_dec (rather than pre_dec) push in