RFR: 8310308: IR Framework: check for type and size of vector nodes [v2]

Emanuel Peter epeter at openjdk.org
Fri Jun 30 08:47:18 UTC 2023


On Fri, 30 Jun 2023 08:32:42 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> test/hotspot/jtreg/compiler/lib/ir_framework/IRNode.java line 249:
>> 
>>> 247:     public static final String ADD_REDUCTION_VD = PREFIX + "ADD_REDUCTION_VD" + POSTFIX;
>>> 248:     static {
>>> 249:         beforeMatchingNameRegex(ADD_REDUCTION_VD, "AddReductionVD");
>> 
>> Shouldn't add reduction nodes only be created in Superword?
>
> Actually, the Vector API can also generate these nodes, and then they would already exist at parsing. I wanted to make these nodes vectorNodes as well, but they do not have a vector output, only a vector input. Hence I cannot match their size.
> We currently do not have any reduction IR tests for the Vector API. I tracked it in [JDK-8310523](https://bugs.openjdk.org/browse/JDK-8310523).
> 
> Actually, the reduction nodes are the only `superWordNodes`. I think we can eventually remove `superWordNodes` completely.

Still, I reverted them to `superWordNodes` for now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14539#discussion_r1247602749


More information about the hotspot-compiler-dev mailing list