RFR: 8329077: C2 SuperWord: Add MoveD2L, MoveL2D, MoveF2I, MoveI2F
Galder Zamarreño
galder at openjdk.org
Thu Jul 24 10:34:38 UTC 2025
I've added support to vectorize `MoveD2L`, `MoveL2D`, `MoveF2I` and `MoveI2F` nodes. The implementation follows a similar pattern to what is done with conversion (`Conv*`) nodes. The tests in `TestCompatibleUseDefTypeSize` have been updated with the new expectations.
Also added a JMH benchmark which measures throughput (the higher the number the better) for methods that exercise these nodes. On darwin/aarch64 it shows:
Benchmark (seed) (size) Mode Cnt Base Patch Units Diff
VectorBitConversion.doubleToLongBits 0 2048 thrpt 8 1168.782 1157.717 ops/ms -1%
VectorBitConversion.doubleToRawLongBits 0 2048 thrpt 8 3999.387 7353.936 ops/ms +83%
VectorBitConversion.floatToIntBits 0 2048 thrpt 8 1200.338 1188.206 ops/ms -1%
VectorBitConversion.floatToRawIntBits 0 2048 thrpt 8 4058.248 14792.474 ops/ms +264%
VectorBitConversion.intBitsToFloat 0 2048 thrpt 8 3050.313 14984.246 ops/ms +391%
VectorBitConversion.longBitsToDouble 0 2048 thrpt 8 3022.691 7379.360 ops/ms +144%
The improvements observed are a result of vectorization. The lack of vectorization in `doubleToLongBits` and `floatToIntBits` demonstrates that these changes do not affect their performance. These methods do not vectorize because of flow control.
I've run the tier1-3 tests on linux/aarch64 and didn't observe any regressions.
-------------
Commit messages:
- Removed unnecessary assert methods
- Adjust IR test after adding Move* vector support
- Delete IR test because it's already covered by other test
- Merge branch 'master' into topic.fp-bits-vector
- Add longBitsToDouble and intBitsToFloat
- Fix test for vectorized and add floatToRawIntBits
- Add basic IR test
- Add JMH benchmark for doubleTo*LongBits
- Support doubleToRawLongBits
- add floatToIntBits benchmark
- ... and 4 more: https://git.openjdk.org/jdk/compare/c68697e1...b6ec784e
Changes: https://git.openjdk.org/jdk/pull/26457/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=26457&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8329077
Stats: 164 lines in 7 files changed: 153 ins; 4 del; 7 mod
Patch: https://git.openjdk.org/jdk/pull/26457.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/26457/head:pull/26457
PR: https://git.openjdk.org/jdk/pull/26457
More information about the core-libs-dev
mailing list