Kyrill Tkachov <kyrylo.tkac...@foss.arm.com> writes: > +;; Helper expander for aarch64_<su>abd<mode>_3 to save the callers > +;; the hassle of constructing the other arm of the MINUS. > +(define_expand "<su>abd<mode>_3" > + [(use (match_operand:VDQ_BHSI 0 "register_operand")) > + (USMAX:VDQ_BHSI (match_operand:VDQ_BHSI 1 "register_operand") > + (match_operand:VDQ_BHSI 2 "register_operand"))] > + "TARGET_SIMD" > + { > + rtx other_arm > + = gen_rtx_<MAX_OPP> (<MODE>mode, operands[1], operands[2]); > + emit_insn (gen_aarch64_<su>abd<mode>_3 (operands[0], operands[1], > + operands[2], other_arm));
Should be indented to the innermost "(" instead. LGTM otherwise, but an AArch6 maintainer should have the final say. Thanks, Richard