RFR: 8366357: C2 SuperWord: refactor VTransformNode::apply with VTransformApplyState [v2]

Emanuel Peter epeter at openjdk.org
Thu Aug 28 13:43:02 UTC 2025


On Thu, 28 Aug 2025 13:29:20 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update src/hotspot/share/opto/vtransform.hpp
>>   
>>   Co-authored-by: Christian Hagedorn <christian.hagedorn at oracle.com>
>
> src/hotspot/share/opto/vtransform.cpp line 737:
> 
>> 735:   // bits in a scalar shift operation. But vector shift does not truncate, so
>> 736:   // we must apply the mask now.
>> 737:   Node* shift_count_masked = new AndINode(shift_count_in, phase->igvn().intcon(_mask));
> 
> Randomly noticed this: You should probably use `phase->intcon()` which also sets root as control`. Same at other places further down. You might want to squeeze that into this patch as well.

Sure, I can do that. It probably does not matter because we set `major_progress` anyway... but better to do it :)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26987#discussion_r2307454827


More information about the hotspot-compiler-dev mailing list