On Mon, Oct 19, 2020 at 12:12:47PM +0000, Christophe Leroy wrote: > From: Mathieu Desnoyers <mathieu.desnoy...@efficios.com> > > The placeholder for instruction selection should use the second > argument's operand, which is %1, not %0. This could generate incorrect > assembly code if the instruction selection for argument %0 ever differs > from argument %1.
"Instruction selection" isn't correct here... "if the memory addressing of operand 0 is a different form from that of operand 1", perhaps? The patch looks fine of course :-) Acked-by: Segher Boessenkool <seg...@kernel.crashing.org> Segher