RFR: 8368977: Provide clear naming for AVX10 identifiers
Jatin Bhateja
jbhateja at openjdk.org
Mon Dec 8 03:33:05 UTC 2025
On Mon, 17 Nov 2025 03:46:50 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. Also, the new AVX10.2 floating point conversion instructions are now used whenever possible to satisfy any related bytecode cast operations. 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`
src/hotspot/cpu/x86/templateTable_x86.cpp line 1605:
> 1603: __ jcc(Assembler::notEqual, L);
> 1604: __ call_VM_leaf(CAST_FROM_FN_PTR(address, SharedRuntime::f2i), 1);
> 1605: }
This change should be part of a seperate PR.
src/hotspot/cpu/x86/templateTable_x86.cpp line 1620:
> 1618: __ jcc(Assembler::notEqual, L);
> 1619: __ call_VM_leaf(CAST_FROM_FN_PTR(address, SharedRuntime::f2l), 1);
> 1620: }
Please restrict this PR to name change related changes only.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28344#discussion_r2596923804
PR Review Comment: https://git.openjdk.org/jdk/pull/28344#discussion_r2596925402
More information about the hotspot-compiler-dev
mailing list