Hi Segher,

on 2020/6/12 上午6:55, Segher Boessenkool wrote:
> Hi!
> 
> On Wed, Jun 10, 2020 at 08:39:19PM +0800, Kewen.Lin wrote:
>> +;; Define optab for vector access with length vectorization exploitation.
>> +(define_expand "lenload<mode>"
>> +  [(match_operand:VEC_A 0 "vlogical_operand")
>> +   (match_operand:VEC_A 1 "memory_operand")
>> +   (match_operand:QI 2 "int_reg_operand")]
> 
> Why this?  gpc_reg_operand will just work, no?  (Even just
> register_operand should, but let's not go there today ;-) )
> 

Good question!  The existing lxvl requires register_operand,
yeah, gpr_reg_operand looks fine too.  I was thinking this
operand for length would only exist in GPR, int_reg_operand
looks more reasonable here?

> Okay for trunk with that change, or with some explanation.  Thanks!
> 

Thanks!

BR,
Kewen

> 
> Segher
> 

Reply via email to