Thanks, the new patch looks great. One minor suggestion: "Qian, Jianhua" <qia...@cn.fujitsu.com> writes: > @@ -1106,7 +1125,45 @@ > mve_move,\ > mve_store,\ > mve_load" > - (const_string "untyped")) > + (cond [(eq_attr "autodetect_type" "alu_shift_lsr_op2") > + (const_string "alu_shift_imm_other") > + (eq_attr "autodetect_type" "alu_shift_asr_op2") > + (const_string "alu_shift_imm_other")
This can be combined into: + (cond [(eq_attr "autodetect_type" "alu_shift_lsr_op2,alu_shift_asr_op2") + (const_string "alu_shift_imm_other") But I think the patch is good to go as-is. Thanks, Richard