[PATCH] Regression fix for PR target/61223

Alexey Merzlyakov alexey.merzlyakov@samsung.com
Thu May 22 13:29:00 GMT 2014


On 20.05.2014 20:24, Ramana Radhakrishnan wrote:
> For now, please revert your original patch and one of Richard or me 
> will try to look at it in the morning. The thumb1 case is slightly 
> more complicated than this.
>
> I don't like this approach as you are introducing the problem again in 
> ARM state
I am not sure, could you provide more details: why do you think ARM 
state is invalid here?

> and leaving the problem as is for Thumb1.
True. I've came up a new draft implementation which supports also thumb1 
(untested). Does it look saner?

Actually, it seems "pop lr" is not really necessary because following by 
"bl _Unwind_Resume @ Never returns" instruction overwrites lr immediately.
Perhaps it may make sense to replace "bl _Unwind_Resume" with "b 
_Unwind_Resume" to keep backtracing also in _Unwind_Resume?

Best regards,
Merzlyakov Alexey
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cxa_fix_fix2.patch
Type: text/x-diff
Size: 1782 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20140522/3eb70801/attachment.bin>


More information about the Gcc-patches mailing list