AW: [11u] RFR: 8248901: Signed immediate support in .../share/assembler.hpp is broken.memory chain

[Doerr, Martin]

Hi Andrew,

we usually add all bug ids to the commit message and make sure that all backported issues get marked as such.
I had forgotten that in this case. Thanks for fixing it.

Best regards,
Martin


Am 23.06.21, 19:49 schrieb "Andrew Hughes" <gnu.andrew at redhat.com>:
On 21:25 Wed 23 Dec     , Doerr, Martin wrote:
> Hi,
>
> I noticed that I need additional fixes for PPC64 and SPARC (see below).
> I've taken the fix for the wrong assertion on PPC64 from upstream and added the missing functions to assembler_sparc.hpp.
>
> New webrev:
> http://cr.openjdk.java.net/~mdoerr/8248901_assembler_11u/webrev.01/
>
> Best regards,
> Martin
>
>

In future, when cherry-picking other fixes for a backport, can you make
sure to credit that bug ID so it is also resolved as backported?

e.g. this is how we ensured all relevant bug IDs were included for the
AArch64 port in 8u:

https://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/f57189b7648d

I've now manually created the backport issue for the PPC fix:

https://bugs.openjdk.java.net/browse/JDK-8215144

Thanks,
--
Andrew :)
Pronouns: he / him or they / them
Senior Free Java Software Engineer
OpenJDK Package Owner
Red Hat, Inc. (http://www.redhat.com)

PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222