On Thu, 22 Oct 2026 07:12:43 GMT, Jatin Bhateja <[email protected]> wrote:

>> Hi All, 
>> 
>> COMPRESS_BITS and EXPAND_BITS advertise compatibility with all integral lane 
>> types, but ByteVector/ShortVector had no implementation and threw 
>> UnsupportedOperationException.
>> 
>> This patch extends COMPRESS_BITS and EXPAND_BITS to the sub-word integral 
>> types byte and short, covering all integral types.
>> 
>> The changes encompass:
>> - Java-side fallback handling for ByteVector and ShortVector.
>> - Inline expander side changes.
>> - C2/VectorSupport type plumbing for T_BYTE/T_SHORT.
>> - Classifying CompressBits/ExpandBits as non-truncating in SuperWord so 
>> subword stores of Integer.compress do not crash debug builds.
>> - Extending Vector API jtreg tests to cover byte and short COMPRESS_BITS and 
>> EXPAND_BITS. 
>> 
>> Please review and share your feedback.
>> 
>> Best Regards,
>> Jatin
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Jatin Bhateja has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains four commits:
> 
>  - Merge branch 'master' of https://github.com/openjdk/jdk into JDK-8392465
>  - Reveiw comments resolution
>  - Review comments resolution
>  - 8392465: [Vector API] VectorOperators.Operator::compatibleWith gives 
> incorrect results for COMPRESS_BITS and EXPAND_BITS

Test results are green but this still needs a second review from someone more 
knowledgeable in the area.

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

PR Comment: https://git.openjdk.org/jdk/pull/33050#issuecomment-6410423665

Reply via email to