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

Reply via email to