Hi! On Wed, Oct 16, 2019 at 09:38:54AM -0400, Michael Meissner wrote: > This patch tweaks the code formatting that I noticed in making the previous > patch for some of the 128-bit mode move instructions. Originally this was > part > of V5 patch #2, but it has been moved to a separate patch.
> @@ -7792,7 +7795,10 @@ (define_insn_and_split "*movtd_64bit_nod > "#" > "&& reload_completed" > [(pc)] > -{ rs6000_split_multireg_move (operands[0], operands[1]); DONE; } > +{ > + rs6000_split_multireg_move (operands[0], operands[1]); > + DONE;} > + > [(set_attr "length" "8,8,8,12,12,8") > (set_attr "max_prefixed_insns" "2") > (set_attr "num_insns" "2,2,2,3,3,2")]) Newline before the } here, and no empty line after. Like all the rest ;-) Okay with that fixed. Thanks! Segher