RFR: 8332119: Incorrect IllegalArgumentException for C2 compiled permute kernel [v2]
Jatin Bhateja
jbhateja at openjdk.org
Tue Jun 4 16:41:12 UTC 2024
On Sun, 2 Jun 2024 15:43:39 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:
>> Currently inline expansion of vector to shuffle conversion simply type casts the vector holding indexes to byte vector[1] where as fallback implementation[2] also wraps the indexes to a valid index range [0, VEC_LEN-1) or generates a -ve index for exceptional / OOB indices.
>>
>> This patch extends the conversion inline expander to match the fall back implementation. This imposes around 20% performance tax on Vector.toShuffle() intrinsic but fixes this functional bug.
>>
>> Kindly review and share your feedback.
>>
>> Best Regards,
>> Jatin
>>
>> PS: Patch also fixes an incorrectness issue reported with [JDK-8332118](https://bugs.openjdk.org/browse/JDK-8332118)
>>
>> [1] https://github.com/openjdk/jdk/blob/master/src/jdk.incubator.vector/share/classes/jdk/incubator/vector/FloatVector.java#L2352
>> [2] https://github.com/openjdk/jdk/blob/master/src/jdk.incubator.vector/share/classes/jdk/incubator/vector/AbstractShuffle.java#L58
>
> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
>
> Review Comments Incorporated.
Hi @TobiHartmann , @vnkozlov , please let me know if it's good to integrate.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/19442#issuecomment-2147966860
More information about the hotspot-compiler-dev
mailing list