On Thu, 12 Feb 2026 02:11:04 GMT, Xiaohong Gong <[email protected]> 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
