On 15 January 2015 at 18:18, Richard Henderson <r...@redhat.com> wrote: > On 12/15/2014 12:41 AM, Zhenqiang Chen wrote: >> +(define_expand "cmp<mode>" >> + [(set (match_operand 0 "cc_register" "") >> + (match_operator:CC 1 "aarch64_comparison_operator" >> + [(match_operand:GPI 2 "register_operand" "") >> + (match_operand:GPI 3 "aarch64_plus_operand" "")]))] >> + "" >> + " >> + operands[1] = gen_rtx_fmt_ee (COMPARE, SELECT_CC_MODE (GET_CODE >> (operands[1]), >> + operands[2], >> + operands[3]), >> + operands[2], operands[3]); >> + " >> +) > > Use { } not "" for the C portion. > > Otherwise ok.
Jiong... this is fine with me. /Marcus