RFR: 8290910: Wrong memory state is picked in SuperWord::co_locate_pack() [v2]

Fei Gao fgao at openjdk.org
Mon Sep 19 07:53:41 UTC 2022


On Tue, 30 Aug 2022 03:32:38 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Fei Gao has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Code style change: add one space
>>   
>>   Change-Id: I2794060ac0f9dbe006e32f202111ee08f09d96a1
>
> Can you show assembler after this fix?
> 
> Would be interest to see results for other interleaving cases:
> 
>     a[i-1] += <expr1>; // similar to your case
>     a[i] += <expr2>;
> 
> 
>     a[i+1] += <expr1>;
>     a[i] += <expr2>;
> 
> 
>     a[i] += <expr1>;
>     a[i-1] += <expr2>;
> 
> 
> Also when `+2` is used instead of `+1`.  Or `+<pack_size>`.

@vnkozlov can I have a review for the new commit? Thanks.

-------------

PR: https://git.openjdk.org/jdk/pull/9898


More information about the hotspot-compiler-dev mailing list