Ah, I didn't realize Loongson was little-endian only. In that case (with mid-end
reductions-via-shifts changes pushed) I don't think I have actually broken
anything, or at least, no MIPS platform that exists :).
However, yes, that would seem a safe bet (and simpler than my linked patch that
provided a BE version too!).
Cheers, Alan
Matthew Fortune wrote:
(for MIPS) https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01481.html,
although I have not been able to test this as there doesn't seem to be
any working MIPS/Loongson hardware in the Compile Farm;
I will post a patch to remove vec_shl and only support vec_shr for
little endian. This is on the basis that loongson only supports
little endian anyway.
I believe this is a safe thing to do regardless of whether your change
is in place or not.
Matthew
