On Fri, 2021-08-27 at 15:38 -0600, Jeff Law wrote:
> 
> 
> On 8/26/2021 10:20 PM, Xi Ruoyao via Gcc-patches wrote:
> > On Thu, 2021-08-26 at 23:56 -0400, YunQiang Su wrote:
> > > gcc/ChangeLog:
> > > 
> > >          PR target/102089
> > >          * config.gcc: MIPS: use N64 ABI by default if the triple
> > > end
> > >            with -gnuabi64, which is used by Debian since 2013.
> > > ---
> > >   gcc/config.gcc | 14 ++++++++++++++
> > >   1 file changed, 14 insertions(+)
> > > 
> > > diff --git a/gcc/config.gcc b/gcc/config.gcc
> > > index 0ff5cac15..0c91be6f3 100644
> > > --- a/gcc/config.gcc
> > > +++ b/gcc/config.gcc
> > > @@ -2553,16 +2553,30 @@ mips*-*-linux*)                         #
> > > Linux MIPS, either endian.
> > >                          target_cpu_default=MASK_SOFT_FLOAT_ABI
> > >                          enable_mips_multilibs="yes"
> > >                          ;;
> > > +               mipsisa64r6*-*-linux-gnuabi64)
> > > +                       default_mips_abi=64
> > > +                       default_mips_arch=mips64r6
> > > +                       enable_mips_multilibs="yes"
> > > +                       ;;
> > >                  mipsisa64r6*-*-linux*)
> > >                          default_mips_abi=n32
> > >                          default_mips_arch=mips64r6
> > >                          enable_mips_multilibs="yes"
> > >                          ;;
> > > +               mipsisa64r2*-*-linux-gnuabi64)
> > > +                       default_mips_abi=64
> > > +                       default_mips_arch=mips64r2
> > > +                       enable_mips_multilibs="yes"
> > > +                       ;;
> > >                  mipsisa64r2*-*-linux*)
> > >                          default_mips_abi=n32
> > >                          default_mips_arch=mips64r2
> > >                          enable_mips_multilibs="yes"
> > >                          ;;
> > > +               mips64*-*-linux-gnuabi64 | mipsisa64*-*-linux-
> > > gnuabi64)
> > > +                       default_mips_abi=64
> > > +                       enable_mips_multilibs="yes"
> > > +                       ;;
> > >                  mips64*-*-linux* | mipsisa64*-*-linux*)
> > >                          default_mips_abi=n32
> > >                          enable_mips_multilibs="yes"
> > LGTM, but I don't have the privilege to approve any change so you'll
> > still need to wait for approval :)
> Yea, but having a second pair of eyes chime in is definitely helpful.
> 
> > 
> > And I think the behavior change should be announced in
> > https://gcc.gnu.org/gcc-12/changes.html, if it's approved.
> Agreed.
> 
> The configure change is approved.

Pushed to trunk as 91f78b67.

Jeff: YunQiang is working on compilers for CIP United, a company
producing MIPS IPs, chips, and toolchains.  Is it possible to grant him
a write-after-approval?

> And a change to the gcc-12 changes.html is pre-approved.

Reply via email to