On 4/30/20 9:28 AM, Stephen Long wrote: > Implements SQSHL/UQSHL, SRSHR/URSHR, and SQSHLU > > Signed-off-by: Stephen Long <stepl...@quicinc.com> > --- > > I'm not too sure about the casting I'm doing willy nilly in the helper > functions, but I think they should be ok. Also, there might be some avenues > for optimization when translating similar to asr_zpzi and lsl_zpzi.
No, the optimization would interfere with the saturation. The casting is ok except for sqshlu. Negative values should saturate to 0, but by passing them to the uqshl helper, the inputs get treated as large unsigned values and saturate to UINT_MAX. You need to use do_suqrshl_bhs for that. r~
