RFR: 8361582: AArch64: Some ConH values cannot be replicated with SVE [v3]
Bhavana Kilambi
bkilambi at openjdk.org
Mon Aug 11 07:54:55 UTC 2025
On Thu, 7 Aug 2025 13:54:39 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Bhavana Kilambi has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Addressed review comments
>
> test/hotspot/jtreg/compiler/c2/aarch64/TestFloat16Replicate.java line 52:
>
>> 50:
>> 51: // Choose FP16_CONST1 which is within the range of [-128 << 8, 127 << 8] and a multiple of 256
>> 52: private static final Float16 FP16_CONST1 = Float16.shortBitsToFloat16((short)512);
>
> Call them `FP16_IN_RANGE` and `FP16_OUT_OF_RANGE`, maybe? Also rename the test cases from `*1`/`*2` to `*InRange`/`*OutOfRange`?
Done
> test/hotspot/jtreg/compiler/c2/aarch64/TestFloat16Replicate.java line 68:
>
>> 66:
>> 67: Generator<Short> gen = G.float16s();
>> 68: IntStream.range(0, LEN).forEach(i -> {input[i] = gen.next();});
>
> Just do a for loop?
Done
> test/hotspot/jtreg/compiler/c2/aarch64/TestFloat16Replicate.java line 79:
>
>> 77: @IR(counts = {IRNode.REPLICATE_HF_IMM8, ">0"},
>> 78: phase = CompilePhase.FINAL_CODE,
>> 79: applyIf = {"MaxVectorSize", ">=32"},
>
> `> 16` then? This matches the comment better.
Done
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26589#discussion_r2265940398
PR Review Comment: https://git.openjdk.org/jdk/pull/26589#discussion_r2265940614
PR Review Comment: https://git.openjdk.org/jdk/pull/26589#discussion_r2265940903
More information about the hotspot-compiler-dev
mailing list