RE: [PATCH,gcc/MIPS] Make loongson3a use fused madd.d

Hi Paul, Your latest version of the patch is now committed. I have been doing some work on the recursive build failure but the issue is complex and involves LRA so I went ahead with committing your change independently. It also turns out that (at least when targeting loongson3a) there are

RE: [PATCH,gcc/MIPS] Make loongson3a use fused madd.d

Hi Paul, Apologies for the delay in responding. > I get the copyright assignment, it's ok for commit. Thanks for going through copyright assignment, I can see you listed and also you have commit access now. Is the trunk build failure still present for you, if it is now resolved then please go

Re: [PATCH,gcc/MIPS] Make loongson3a use fused madd.d

---Original Message- >> From: Paul Hua [mailto:paul.hua...@gmail.com] >> Sent: 17 November 2016 03:01 >> To: Matthew Fortune >> Cc: gcc-patches@gcc.gnu.org; catherine_mo...@mentor.com >> Subject: Re: [PATCH,gcc/MIPS] Make loongson3a use fused madd.d >> &g

RE: [PATCH,gcc/MIPS] Make loongson3a use fused madd.d

il.com] > Sent: 17 November 2016 03:01 > To: Matthew Fortune > Cc: gcc-patches@gcc.gnu.org; catherine_mo...@mentor.com > Subject: Re: [PATCH,gcc/MIPS] Make loongson3a use fused madd.d > > ping... > > On Thu, Nov 3, 2016 at 7:58 PM, Paul Hua <paul.hua...@gmail.com> wrote: >

Re: [PATCH,gcc/MIPS] Make loongson3a use fused madd.d

ping... On Thu, Nov 3, 2016 at 7:58 PM, Paul Hua wrote: > Hi Matthew, > > Thanks for your comments, update the patch. > > *** gcc/ChangeLog *** > > 2016-11-03 Chenghua Xu > > * config/mips/mips.h (ISA_HAS_FUSED_MADD4): Enable for >

Re: [PATCH,gcc/MIPS] Make loongson3a use fused madd.d

Hi Matthew, Thanks for your comments, update the patch. *** gcc/ChangeLog *** 2016-11-03 Chenghua Xu * config/mips/mips.h (ISA_HAS_FUSED_MADD4): Enable for TARGET_LOONGSON_3A. (ISA_HAS_UNFUSED_MADD4): Exclude TARGET_LOONGSON_3A. Thanks, Paul On

RE: [PATCH,gcc/MIPS] Make loongson3a use fused madd.d

Paul Hua writes: > Loongson3a has 4 operand fused madd instrcution. This patch set > loongson3a use fused madd.d. Hi Paul, Thanks for the fix. I was vaguely aware that this was wrong for loongson-3a but never confirmed it. I suspect this change is mechanical enough that