On 5/10/23 1:06 PM, Carl Love wrote:
> - void __builtin_altivec_tr_stxvrhx (vsq, signed long, signed int *);
> + void __builtin_altivec_tr_stxvrhx (vsq, signed long, signed short *);
> TR_STXVRHX vsx_stxvrhx {stvec}
>
> - void __builtin_altivec_tr_stxvrwx (vsq, signed long, signed short *);
> + void __builtin_altivec_tr_stxvrwx (vsq, signed long, signed int *);
> TR_STXVRWX vsx_stxvrwx {stvec}
In my estimation, these two changes are "obvious" fixes.
> + void __builtin_vec_xst_trunc (vsq, signed long long, signed long *);
> + TR_STXVRLX TR_STXVRLX_S
> + void __builtin_vec_xst_trunc (vuq, signed long long, unsigned long *);
> + TR_STXVRLX TR_STXVRLX_U
Not a comment on these two changes, and not a request to expand this
specific patch, but I believe I saw other built-ins that were missing
signed long */unsigned long * versions where they could/should accept
them. Can you double-check whether there are other built-ins that
need similar changes and if so, please post a separate patch to fix
those as well? Thanks.
Peter
