On Wed, Jan 13, 2021 at 08:26:49AM +0100, Richard Biener wrote:
> +  if (op1 && op0 != op1)
> +    op1 = force_reg (vmode, op1);
> 
> code (presumably to handle RTX sharing here)?

That could be actually simplified, incrementally e.g. to:
   if (op0)
     {
       rtx nop0 = force_reg (vmode, op0);
       if (op0 == op1)
         op1 = nop0;
       op0 = nop0;
     }
-  if (op1 && op0 != op1)
+  if (op1)
     op1 = force_reg (vmode, op1);

(because the outer force_reg in force_reg (vmode, force_reg (vmode, X))
just returns its argument).

        Jakub

Reply via email to