On 11/27/23 10:53, Stefan Schulze Frielinghaus wrote: > Commit 248df13b966f46649e16dc3c8c92b263790ef503 restricted the rotate > count to immediates. Although the documentation of vec_rli (Vector > Element Rotate Left Immediate) can be read as if it where restricted to > immediates, this is not the case. Thus, revert this commit. > > In order to finally allow register operands, the rotate count must be of > type unsigned char since the expander expects it to be of mode QI. The > previously used type unsigned integer worked out for immediates since > those are of VOID mode anyway. > > Bootstrapped and regtested on s390. Ok for mainline? > > gcc/ChangeLog: > > * config/s390/s390-builtin-types.def: Remove types. > * config/s390/s390-builtins.def (O_U64): Remove 64-bit literal support. > Don't restrict s390_vec_rli and s390_verll[bhfg] to immediates. > * config/s390/s390.cc (s390_const_operand_ok): Remove 64-bit > literal support.
Ok, Thanks! Andreas
