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