RFR: 8358521: Optimize vector operations by reassociating broadcasted inputs [v2]

[Jatin Bhateja]

On Thu, 12 Feb 2026 02:11:04 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:

>> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review comments resolutions
>
> src/hotspot/share/opto/vectornode.cpp line 1303:
> 
>> 1301: //
>> 1302: Node* VectorNode::reassociate_vector_operation(PhaseGVN* phase) {
>> 1303:   if (is_commutative_vector_operation(Opcode()) && is_integral_type(vect_type()->element_basic_type())) {
> 
> Suggestions
> 
>   // Only integral vector operations support reassociate ...
>   if (is_integral_type(vect_type()->element_basic_type())) {
>     return nullptr;
>   }
> 
>   // Only commutative vector operations supports ...
>   if (!is_commutative_vector_operation(Opcode()) {
>     return nullptr;
>   }

Addressed!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25617#discussion_r2796806875