Hi! On Sun, Sep 12, 2021 at 10:50:17AM -0500, Bill Schmidt wrote: > On 9/8/21 1:42 AM, HAO CHEN GUI wrote: > >+;; Vector reverse elements for V2DI V2DF > >+(define_expand "altivec_vreve<mode>2" > >+ [(set (match_operand:VEC_64 0 "register_operand" "=v") > >+ (unspec:VEC_64 [(match_operand:VEC_64 1 "register_operand" > >"v")] > >+ UNSPEC_VREVEV))] > >+ "TARGET_ALTIVEC"
(Your quoted text is mangled again) > "TARGET_VSX" for V2DI and V2DF. (This is the only good reason for > splitting this into two patterns; you need different criteria.) The *good* reason for splitting the pattern is they have completely different expansions as well. Which is why I asked for it. (I'll review the patch later). Segher
