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