Hi Bill,

On Thu, Feb 16, 2017 at 02:16:02PM -0600, Bill Schmidt wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79261 records that the interface
> vec_xxpermdi isn't implemented in a bi-endian fashion; instead, it produces
> results appropriate for big-endian vector element numbering even when run on
> a little endian machine.  This is not part of the "official vector API" from
> the ELFv2 ABI document, but should still have appropriate bi-endian behavior.

Maybe this needs adding (or updating) some documentation?

> +;; Special version of xxpermdi that retains big-endian semantics.
> +(define_expand "vsx_xxpermdi_<mode>_be"
> +  [(match_operand:VSX_L 0 "vsx_register_operand" "")
> +   (match_operand:VSX_L 1 "vsx_register_operand" "")
> +   (match_operand:VSX_L 2 "vsx_register_operand" "")
> +   (match_operand:QI 3 "u5bit_cint_operand" "")]

Please remove the "".

Okay with that and perhaps some doc changes.  Thanks,


Segher

Reply via email to