[Bug target/88656] [7/8/9/10 Regression] lr clobbered by thumb prologue before __builtin_return_address(0) reads from it

2019-05-25 Thread gerd at egidy dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88656

gerd at egidy dot de changed:

   What|Removed |Added

 CC||gerd at egidy dot de

--- Comment #2 from gerd at egidy dot de ---
Could it be that this is a duplicate of bug 88167?

I compiled a gcc 7.4.0 patched with the fix for 88167 and now get this result:

push{r4, lr}
mov r3, r8
mov r4, r9
push{r3, r4}
mov r0, lr

The patch for bug 88167 seems to be just in trunk for now. As the problem is a
regression in all releases till gcc 7 I'd prefer if it could be backported into
the corresponding branches.

[Bug target/88656] [7/8/9/10 Regression] lr clobbered by thumb prologue before __builtin_return_address(0) reads from it

2019-05-28 Thread gerd at egidy dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88656

--- Comment #4 from gerd at egidy dot de ---
(In reply to Richard Earnshaw from comment #3)
> A regression is not a bug that applies in all previous releases.  A
> regression is where something worked in some previous releases but does not
> work now.

using __builtin_return_address(0) as described in the bug report worked in gcc
5.4.0 and before. To be more specific, it worked before the fix for bug 77933
was incorporated. So to me this is a regression.

[Bug target/88656] [7/8/9/10 Regression] lr clobbered by thumb prologue before __builtin_return_address(0) reads from it

2019-05-28 Thread gerd at egidy dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88656

--- Comment #6 from gerd at egidy dot de ---
(In reply to Richard Earnshaw from comment #5)
> then please update the known-to-work field accordingly.

It seems I'm missing the rights to do so, could someone with the rights please
do that.