RFR: 8358521: Optimize vector operations by reassociating broadcasted inputs [v3]
Xiaohong Gong
xgong at openjdk.org
Mon Feb 23 07:07:15 UTC 2026
On Tue, 10 Feb 2026 05:33:28 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:
>> src/hotspot/share/opto/vectornode.cpp line 1317:
>>
>>> 1315: }
>>> 1316: return nullptr;
>>> 1317: }
>>
>> Reassociation can be a separate optimization for vector nodes to me. Do you think it's better to split it as a separate change following the broadcast optimization? We can add specific tests for it.
>
> Reassociation are only applicable to integral vector operation, for vector API we only use fp-fast model semantics for reductions of floating point vector, I have now added tests to cover all possible transformations so that we can include both associated transforms in same patch.
I still keep my point that separating the reassociation as a separate PR for better maintain. WDYT?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25617#discussion_r2839345895
More information about the core-libs-dev
mailing list