On Fri, 1 Aug 2025 11:58:03 GMT, Quan Anh Mai <[email protected]> wrote:
> VectorNode::is_reinterpret_opcode returns true for Op_ReinterpretHF2S and
> Op_ReinterpretS2HF, which are very similar to the nodes in this PR, can you
> add these nodes to that method instead?
You're suggesting to modify `is_reinterpret_opcode` to be like this, and call
that instead of `is_move_opcode`, right?
bool VectorNode::is_reinterpret_opcode(int opc) {
switch (opc) {
case Op_ReinterpretHF2S:
case Op_ReinterpretS2HF:
case Op_MoveF2I:
case Op_MoveD2L:
case Op_MoveL2D:
case Op_MoveI2F:
return true;
default:
return false;
}
}
-------------
PR Comment: https://git.openjdk.org/jdk/pull/26457#issuecomment-3153512422
