[PATCH] Fix .cfi_* directive skip over >= 64KB (PR gas/10255)

H.J. Lu hjl.tools@gmail.com
Tue Jun 9 13:41:00 GMT 2009


On Tue, Jun 9, 2009 at 6:32 AM, Jakub Jelinek<jakub@redhat.com> wrote:
> On Tue, Jun 09, 2009 at 06:27:37AM -0700, H.J. Lu wrote:
>> > I don't think a memory write is something we should try hard to avoid, what
>> > matters how it is maintanable.  IMHO leaving the fragment uninitialized
>> > is a bad idea, but doing what you wrote above in eh_frame_convert_frag
>>
>> It is uninitialized because we don't know what we will put in there.
>> If we want to initialize it, I think it should be initialized to something
>> invalid and verify it is valid when we output it. Otherwise, initialize
>> it may still generate bad output.
>
> I don't understand it.  We generate DW_CFA_advance_loc4 by default, and if
> possible optimize it to something shorter.  So DW_CFA_advance_loc4 is the
> right default...
>

That is true only because of the logic in  eh_frame_convert_frag.


-- 
H.J.



More information about the Binutils mailing list