On 11/13/18 12:49 PM, Richard Henderson wrote: > On 11/13/18 5:38 PM, Peter Bergner wrote: >> On 11/13/18 2:53 AM, Eric Botcazou wrote: >>> Superfluous semi-colon. Given that the function returns an operand, its >>> name >>> is IMO misleading, so maybe [get_]operand_for_simple_move_operator. >> >> Fixed and renamed function to operand_for_simple_move_operator. > > Would not operand_for_swap_move_operator be better? This is not a "simple > move", it is something that requires swapping the words of the operand. > (Presumably one could think of other operators that generate a swap, and match > them here. I can't think of another one off the top of my head though.)
That's fine with me. Peter