On Fri, Jan 28, 2022 at 11:50:23AM -0600, Bill Schmidt wrote:
> These built-ins were misimplemented as always having big-endian semantics.

>       PR target/95082
>       * config/rs6000/rs6000-builtin.cc (rs6000_expand_builtin): Handle
>       endianness for vclzlsbb and vctzlsbb.
>       * config/rs6000/rs6000-builtins.def (VCLZLSBB_V16QI): Change
>       default pattern and indicate a different pattern will be used for
>       big endian.
>       (VCLZLSBB_V4SI): Likewise.
>       (VCLZLSBB_V8HI): Likewise.
>       (VCTZLSBB_V16QI): Likewise.
>       (VCTZLSBB_V4SI): Likewise.
>       (VCTZLSBB_V8HI): Likewise.

Please don't break lines early?  Changelog lines are one tab followed
by 72 chars.

The patch is fine though.  Thanks!


Segher

Reply via email to