RFR: 8368977: Provide clear naming for AVX10 identifiers [v2]

Jatin Bhateja jbhateja at openjdk.org
Wed Dec 10 10:07:28 UTC 2025


On Mon, 8 Dec 2025 21:47:16 GMT, Mohamed Issa <missa at openjdk.org> wrote:

>> This is a simple change that renames all AVX10 identifiers to explicitly show which sub-versions are in use. Right now, AVX10.2 is the only case to worry about. The JTREG tests listed below were used to verify correctness with the recommended JVM options mentioned in corresponding source files. Each test included runs through emulation with AVX10.2 enabled and disabled to exercise all possible paths. All modifications and tests used [OpenJDK v26-b24](https://github.com/openjdk/jdk/releases/tag/jdk-26%2B24) as the baseline build.
>> 
>> 1. `jtreg:test/hotspot/jtreg/compiler/codegen/TestByteDoubleVect.java`
>> 2. `jtreg:test/hotspot/jtreg/compiler/codegen/TestByteFloatVect.java`
>> 3. `jtreg:test/hotspot/jtreg/compiler/codegen/TestIntDoubleVect.java`
>> 4. `jtreg:test/hotspot/jtreg/compiler/codegen/TestIntFloatVect.java`
>> 5. `jtreg:test/hotspot/jtreg/compiler/codegen/TestLongDoubleVect.java`
>> 6. `jtreg:test/hotspot/jtreg/compiler/codegen/TestLongFloatVect.java`
>> 7. `jtreg:test/hotspot/jtreg/compiler/codegen/TestShortDoubleVect.java`
>> 8. `jtreg:test/hotspot/jtreg/compiler/codegen/TestShortFloatVect.java`
>> 9. `jtreg:test/hotspot/jtreg/compiler/floatingpoint/ScalarFPtoIntCastTest.java`
>> 10. `jtreg:test/hotspot/jtreg/compiler/vectorapi/VectorFPtoIntCastTest.java`
>> 11. `jtreg:test/hotspot/jtreg/compiler/vectorization/runner/ArrayTypeConvertTest.java`
>> 12. `jtreg:test/jdk/jdk/incubator/vector/Double64VectorTests.java`
>> 13. `jtreg:test/jdk/jdk/incubator/vector/Double128VectorTests.java`
>> 14. `jtreg:test/jdk/jdk/incubator/vector/Double256VectorTests.java`
>> 15. `jtreg:test/jdk/jdk/incubator/vector/Double512VectorTests.java`
>> 16. `jtreg:test/jdk/jdk/incubator/vector/DoubleMaxVectorTests.java`
>> 17. `jtreg:test/jdk/jdk/incubator/vector/Float64VectorTests.java`
>> 18. `jtreg:test/jdk/jdk/incubator/vector/Float128VectorTests.java`
>> 19. `jtreg:test/jdk/jdk/incubator/vector/Float256VectorTests.java`
>> 20. `jtreg:test/jdk/jdk/incubator/vector/Float512VectorTests.java`
>> 21. `jtreg:test/jdk/jdk/incubator/vector/FloatMaxVectorTests.java`
>> 22. `jtreg:test/hotspot/jtreg/compiler/vectorization/TestFloat16VectorOperations.java`
>> 23. `jtreg:test/hotspot/jtreg/compiler/c2/irTests/TestFloat16ScalarOperations.java`
>
> Mohamed Issa has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Remove changes that affect functionality

LGTM, naming now follows the AVX10 versions sheme where an instruction supported by a version is guaranteed to be supported by all the version above it.

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

Marked as reviewed by jbhateja (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/28344#pullrequestreview-3561706798


More information about the hotspot-dev mailing list