RFR: 8342393: Promote commutative vector IR node sharing [v14]

[Jatin Bhateja]

On Thu, 23 Jan 2025 12:37:52 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

> > > I'm a little sad that `is_commutative_vector_operation` does not mean that it is actually commutative now. Rather, it means: "it is commutative if it is not predicated"
> > 
> > 
> > Hey @eme64 , Your comments are addressed.
> 
> What was you answer to my concern that `is_commutative_vector_operation` does not mean that it is actually commutative (with the conter example of predicated case)?
> 
> I'm still confused about your comment: does `BLEND` not have 3 operands? You closed my thread as resolved without really answering all of my questions - I'm still confused 😅

Hi @eme64 , let me think about this solution again. Flags-based solutions are generally troublesome in the long run.  Although we do have a follow-up RFE, but I feel its good to spend a little more time here as it fixes a very important gap.

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

PR Comment: https://git.openjdk.org/jdk/pull/22863#issuecomment-2609876680