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.  And a change to the gcc-12 changes.html is pre-approved.

jeff

Reply via email to