RFR: 8366702: C2 SuperWord: refactor VTransform vector nodes [v2]

Emanuel Peter epeter at openjdk.org
Mon Sep 8 08:14:54 UTC 2025


On Mon, 8 Sep 2025 07:25:47 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review comment implemented
>
> src/hotspot/share/opto/vtransform.hpp line 600:
> 
>> 598: 
>> 599: // Bundle the information needed for vector nodes.
>> 600: class VTransformVectorNodePrototype : public StackObj {
> 
> Prototype sounds like actually having something concrete, not fully set up or just something to copy/clone from as a starting point. But IIUC, this class just serves as a holder class for some information. How about naming it `Prototype` -> `Properties`?

I like the idea :)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27056#discussion_r2329474779


More information about the hotspot-compiler-dev mailing list