This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Patch: PR debug/45865
On Sun, Oct 10, 2010 at 4:06 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Sun, Oct 10, 2010 at 6:50 AM, Jason Merrill <jason@redhat.com> wrote:
>> On 10/09/2010 10:41 PM, H.J. Lu wrote:
>>>
>>> Does this patch make senses?
>>
>> If the problem is that there's a return jump that isn't so marked, that
>> seems to be the thing to fix, rather than assume that any jump within an
>> epilogue is a return.
>>
>
> That function has multiple epilogues:
>
> ?epilogue 1
> ....
> label1:
> ?...
> ?return
>
> ?epilogue 2
> ?...
> ?jump label1
> ?epilogue 3
>
> dwarf2out_cfi_begin_epilogue fails to properly handle epilogue 2. It
> doesn't stop at "jump label1" and treats epilogue 3 is the part of
> epilogue 2. My patch makes dwarf2out_cfi_begin_epilogue to stop
> at "jump label1".
>
Bernd, should your ifcvt/crossjump change mark "jump label1"
in epilogue as return jump?
--
H.J.