RFR: 8365911: AArch64: Fix encoding error in sve_cpy for negative floats [v2]
Andrew Haley
aph at openjdk.org
Wed Sep 3 08:30:43 UTC 2025
On Wed, 3 Sep 2025 08:23:32 GMT, erifan <duke at openjdk.org> wrote:
>> src/hotspot/cpu/aarch64/assembler_aarch64.hpp line 3819:
>>
>>> 3817: if (isFloat) {
>>> 3818: assert(T != B, "invalid size");
>>> 3819: assert((imm8 >> 8) == 0, "invalid immediate");
>>
>> Suggestion:
>>
>> assert((imm8 & 0xff) == 0, "invalid immediate");
>>
>> To match line 3819.
>
> This may not be the case, `imm8 >> 8` doesn't equal to `imm8 & 0xff`
What is the range of values you're trying to test?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26951#discussion_r2318194266
More information about the hotspot-compiler-dev
mailing list