RFR: 8333840: C2 SuperWord: wrong result for MulAddS2I when inputs permuted [v3]

Emanuel Peter epeter at openjdk.org
Thu Jun 13 06:52:13 UTC 2024


On Wed, 12 Jun 2024 15:26:54 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   more refactoring
>
> src/hotspot/share/opto/superword.cpp line 2837:
> 
>> 2835: // Inputs:                 1    2    3    4
>> 2836: // Offsets:                0    0    1    1
>> 2837: //   v = MulAddS2I(a, b) = a0 * b0 + a1 + b1
> 
> Typo, should be `a1 * b1`

fixed!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19619#discussion_r1637646061


More information about the hotspot-compiler-dev mailing list