RFR: 8329077: C2 SuperWord: Add MoveD2L, MoveL2D, MoveF2I, MoveI2F [v3]

Galder Zamarreño galder at openjdk.org
Mon Aug 11 14:09:19 UTC 2025


On Sun, 10 Aug 2025 05:23:23 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Galder Zamarreño has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Check at the very least that auto vectorization is supported
>
> src/hotspot/share/opto/superword.cpp line 1635:
> 
>> 1633:     } else if (VectorNode::is_convert_opcode(opc)) {
>> 1634:       retValue = VectorCastNode::implemented(opc, size, velt_basic_type(p0->in(1)), velt_basic_type(p0));
>> 1635:     } else if (VectorNode::is_reinterpret_opcode(opc)) {
> 
> How does this affect `Op_ReinterpretHF2S` that is also in `VectorNode::is_reinterpret_opcode`?
> I'm afraid that we need to test this with hardware or Intel's SDE, to make sure we have it running on a VM that actually supports Float16. Otherwise these instructions may not be used, and hence not tested right.
> 
> @galderz Can you run the relevant tests?

Would you run specific tiers in those platforms? Just hotspot compiler? Or individual tests such as `ConvF2HFIdealizationTests` and `TestFloat16ScalarOperations`?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26457#discussion_r2266889792


More information about the core-libs-dev mailing list