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

[Aleksey Shipilev]

On 2/20/20 5:26 AM, Andrew Hughes wrote:
> 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.

Thanks, added the tags.

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

Yes, that is my plan, and that is why issue has 8-aarch64 as affected version, so our tracking shows
this as missing backport in aarch64-port. I would prefer to have the test in from upstream before
transplanting the hunk to aarch64.

-- 
Thanks,
-Aleksey