On 4/23/21 6:58 PM, Segher Boessenkool wrote:
On Fri, Apr 23, 2021 at 06:24:07PM -0400, Michael Meissner wrote:
On Thu, Apr 22, 2021 at 05:56:32PM -0500, Segher Boessenkool wrote:
As Will says, it looks like the ELFv2 version has the same bug. Please
fix that the same way.
Yes it has the same bug. However in practice it would never be hit, since this
bug is 32-bit, and we only build 64-bit systems with ELF v2. I did fix it.
Hrm, in that case, why do we have that code at all?!
Okay for trunk. Okay for backport to 11 when that branch opens again.
Does this need more backports? (Those should follow after 11 of
course).
Bill mentioned we may want to backport this to earlier branches before they are
frozen. Tulio, are backports to earlier revisions important?
Well, the bug has been there since the original commit to (then)
tramp.asm, which was 25 years ago, and only now people noticed ;-)
We should have a backport to GCC 11 at least. Older is up to you (and
Tulio).
This was reported to us as a compatibility problem with Clang that was
holding up porting a language runtime to Power. Since this is very
obviously a bug, I would like to be aggressive about backporting it to
previous releases to avoid any other such problems. Thanks for considering!
Bill
Segher
