RFR: 8363989: AArch64: Add missing backend support of VectorAPI expand operation [v2]

erifan duke at openjdk.org
Fri Sep 5 06:30:34 UTC 2025


On Thu, 4 Sep 2025 08:01:40 GMT, erifan <duke at openjdk.org> wrote:

>> src/hotspot/cpu/aarch64/c2_MacroAssembler_aarch64.cpp line 2819:
>> 
>>> 2817:   subv(dst, size, tmp2, tmp1);
>>> 2818:   // dst = 0 0 8 7 0 0 6 5 0 0 4 3 0 0 2 1
>>> 2819:   tbl(dst, size, src, 1, dst);
>> 
>> It would make it a little easier to read the example if the numbers were aligned.
>> Now the minus sign disrupts that a little. Maybe leave 2 spaces if the number is positive?
>
> Make sense, I'll update it in the following commit.

Done, thanks!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26740#discussion_r2324218817


More information about the hotspot-compiler-dev mailing list