On Wed, 22 Nov 2023 02:18:32 GMT, Eric Liu <e...@openjdk.org> wrote: >> src/hotspot/cpu/aarch64/c2_MacroAssembler_aarch64.cpp line 1412: >> >>> 1410: _sve_xunpk(is_unsigned, /* is_high */ false, dst, S, dst); >>> 1411: _sve_xunpk(is_unsigned, /* is_high */ false, dst, D, dst); >>> 1412: break; >> >> Why is this change here? It doesn't do anything. Does it? > > `is_unsigned` is also used in this function. It is used in VectorUCastNode > for zero extending.
Ah, I see. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16670#discussion_r1401721522
