[8u] RFR 8181872: C1: possible overflow when strength reducing integer multiply by constant

[Andrew Hughes]

On 19/02/2020 12:18, Aleksey Shipilev wrote:
> Replying with bug id in the subject, for archival reasons.
> 
> On 2/19/20 1:17 PM, Aleksey Shipilev wrote:
>> Original fix:
>>   https://bugs.openjdk.java.net/browse/JDK-8181872
>>   https://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/16c9c159df90
>>
>> Patch applies cleanly, except that aarch64 and arm is not in 8u, so those hunks are ignored.
>>
>> 8u webrev:
>>   https://cr.openjdk.java.net/~shade/8181872/webrev.8u.01/
>>
>> Testing: tier1, affected test (it passes even without the fix, probably because my toolchain does
>> not break it)
> 

Looks fine.

I assume you'll apply the AArch64 fix after this makes it into
aarch64-port/shenandoah-jdk8u?

Thanks,
-- 
Andrew :)

Senior Free Java Software Engineer
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