Karl Meakin <karl.mea...@arm.com> writes:
> @@ -729,30 +729,31 @@ (define_expand "cbranch<mode>4"
>               (match_operator 0 "aarch64_comparison_operator"
>                [(match_operand:GPF_F16 1 "register_operand")
>                 (match_operand:GPF_F16 2 "aarch64_fp_compare_operand")])
> -             (label_ref (match_operand 3 "" ""))
> +             (label_ref (match_operand 3))
>               (pc)))]
>    ""
> -  "
> +  {
>    operands[1] = aarch64_gen_compare_reg (GET_CODE (operands[0]), operands[1],
>                                        operands[2]);
>    operands[2] = const0_rtx;
> -  "
> +  }
>  )

I was going to make this change locally before pushing, since it's
so trivial, but: could you indent the code between the braces by two
extra spaces while you're there?  The current indentation seems
"more wrong" with braces than quotes.

Thanks,
Richard

Reply via email to