Hi Juzhe,

LGTM once the middle-end parts are in.  Just tiny nits.
Nothing that would warrant a V2, though.

> +;; =========================================================================
> +;; == Stried Load/Store

missing a 'd' here.
 
> +(define_predicate "vector_stride_extension_operand"
> +  (ior (and (match_operand 0 "immediate_operand")
> +            (match_test "Pmode == DImode"))
> +       (and (match_operand 0 "const_0_operand")
> +            (match_test "Pmode == SImode"))))
> +

This could use a comment why we allow only sign extension
for 32 bit.  Also the linter complains about spaces vs tabs.

Regards
 Robin

Reply via email to