Sorry for the late response.
I've checked the patch on gcc-9 and gcc-8 branches with mips{el}-linux-gnu.
There are no new regressions, as expected.
Is this change still eligible for back-port ?
________________________________
From: Jeff Law <l...@redhat.com>
Sent: Friday, May 31, 2019 11:41:35 PM
To: Dragan Mladjenovic; gcc-patches@gcc.gnu.org
Cc: Matthew Fortune
Subject: Re: [MIPS] Fix for the wrong argument sequence in MSA builtin for
FMADD/MADDV family.
On 5/22/19 12:13 AM, Dragan Mladjenovic wrote:
> Hi all,
>
> This patch makes the behavior of __builtin_msa_ fmadd/fmsub/maddv match the
> expected one. While not a recent regression, it would be nice if this can
> be back-ported in order to minimize the fragmentation.
>
> Best regards,
>
> Dragan
>
> gcc/ChangeLog:
>
> 2019-05-20 Prachi Godbole <prachi.godb...@imgtec.com>
> Robert Suchanek <robert.sucha...@mips.com>
>
> * gcc/config/mips/mips.c (mips_expand_builtin_insn): Swap the 1st
> and 3rd operands of the fmadd/fmsub/maddv builtin.
>
> gcc/testsuite/ChangeLog:
>
> 2019-05-20 Dragan Mladjenovic <dmladjeno...@wavecomp.com>
>
> * gcc.target/mips/msa-fmadd.c: New.
THanks. I've installed this one the trunk.
Out of an abundance of caution I have not installed it on any of the
release branches. Feel free to ping me in a week or so and if there
haven't been any issues I'll back port it.
jeff
