RFR: 8366357: C2 SuperWord: refactor VTransformNode::apply with VTransformApplyState [v3]
Emanuel Peter
epeter at openjdk.org
Thu Aug 28 14:50:46 UTC 2025
On Thu, 28 Aug 2025 14:42:46 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:
>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>>
>> For Christian: use phase->intcon instead
>
> src/hotspot/share/opto/superword.cpp line 3072:
>
>> 3070:
>> 3071: // 1.1: con
>> 3072: Node* xbic = phase()->intcon(is_sub ? -con : con);
>
> What changed that you need/want `PhaseIdealLoop*` here instead of `PhaseIterGVN*`?
@chhagedorn Suggested it here: https://github.com/openjdk/jdk/pull/26987#discussion_r2307425745
That way, we set the control of the new constant. It was not strictly necessary, but good practice.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26987#discussion_r2307649223
More information about the hotspot-compiler-dev
mailing list