diff mbox series

testsuite, rs6000: Remove some checks with aix[456]

Message ID ead493e5-caba-6678-99c9-84a54529f976@linux.ibm.com
State New
Headers
Series testsuite, rs6000: Remove some checks with aix[456] |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_gcc_build--master-arm fail Patch failed to apply
linaro-tcwg-bot/tcwg_gcc_build--master-aarch64 fail Patch failed to apply

Commit Message

Kewen.Lin May 8, 2024, 6:36 a.m. UTC 
  Hi,

Since r12-75-g0745b6fa66c69c aix6 support had been dropped,
so we don't need to check for aix[456].* when testing, this
patch is to remove such checks.

Regtested on powerpc64-linux-gnu P8/P9 and
powerpc64le-linux-gnu P9 and P10.

I'm going to push this soon if no objections.

BR,
Kewen
-----

gcc/testsuite/ChangeLog:

	* lib/target-supports.exp
	(check_effective_target_powerpc_altivec_ok): Remove checks for
	aix[456].*
	(check_effective_target_powerpc_p9modulo_ok): Likewise.
	(check_effective_target_powerpc_float128_sw_ok): Likewise.
	(check_effective_target_powerpc_float128_hw_ok): Likewise.
	(check_effective_target_powerpc_vsx_ok): Likewise.
---
 gcc/testsuite/lib/target-supports.exp | 29 ---------------------------
 1 file changed, 29 deletions(-)

--
2.39.1

Comments

David Edelsohn May 8, 2024, 2:54 p.m. UTC | #1 
On Wed, May 8, 2024 at 2:36 AM Kewen.Lin <linkw@linux.ibm.com> wrote:

> Hi,
>
> Since r12-75-g0745b6fa66c69c aix6 support had been dropped,
> so we don't need to check for aix[456].* when testing, this
> patch is to remove such checks.
>
> Regtested on powerpc64-linux-gnu P8/P9 and
> powerpc64le-linux-gnu P9 and P10.
>

Okay


>
> I'm going to push this soon if no objections.
>
> BR,
> Kewen
> -----
>
> gcc/testsuite/ChangeLog:
>
>         * lib/target-supports.exp
>         (check_effective_target_powerpc_altivec_ok): Remove checks for
>         aix[456].*
>         (check_effective_target_powerpc_p9modulo_ok): Likewise.
>         (check_effective_target_powerpc_float128_sw_ok): Likewise.
>         (check_effective_target_powerpc_float128_hw_ok): Likewise.
>         (check_effective_target_powerpc_vsx_ok): Likewise.
> ---
>  gcc/testsuite/lib/target-supports.exp | 29 ---------------------------
>  1 file changed, 29 deletions(-)
>
> diff --git a/gcc/testsuite/lib/target-supports.exp
> b/gcc/testsuite/lib/target-supports.exp
> index 3a55b2a4159..16dc2766850 100644
> --- a/gcc/testsuite/lib/target-supports.exp
> +++ b/gcc/testsuite/lib/target-supports.exp
> @@ -6963,11 +6963,6 @@ proc check_effective_target_powerpc_altivec_ok { } {
>      # Paired Single, then not ok
>      if { [istarget powerpc-*-linux*paired*] } { return 0 }
>
> -    # AltiVec is not supported on AIX before 5.3.
> -    if { [istarget powerpc*-*-aix4*]
> -        || [istarget powerpc*-*-aix5.1*]
> -        || [istarget powerpc*-*-aix5.2*] } { return 0 }
> -
>      # Return true iff compiling with -maltivec does not error.
>      return [check_no_compiler_messages powerpc_altivec_ok object {
>         int dummy;
> @@ -6980,12 +6975,6 @@ proc check_effective_target_powerpc_p9modulo_ok { }
> {
>      if { ([istarget powerpc*-*-*]
>           && ![istarget powerpc-*-linux*paired*])
>          || [istarget rs6000-*-*] } {
> -       # AltiVec is not supported on AIX before 5.3.
> -       if { [istarget powerpc*-*-aix4*]
> -            || [istarget powerpc*-*-aix5.1*]
> -            || [istarget powerpc*-*-aix5.2*] } {
> -           return 0
> -       }
>         return [check_no_compiler_messages powerpc_p9modulo_ok object {
>             int main (void) {
>                 int i = 5, j = 3, r = -1;
> @@ -7116,12 +7105,6 @@ proc check_effective_target_powerpc_float128_sw_ok
> { } {
>      if { ([istarget powerpc*-*-*]
>           && ![istarget powerpc-*-linux*paired*])
>          || [istarget rs6000-*-*] } {
> -       # AltiVec is not supported on AIX before 5.3.
> -       if { [istarget powerpc*-*-aix4*]
> -            || [istarget powerpc*-*-aix5.1*]
> -            || [istarget powerpc*-*-aix5.2*] } {
> -           return 0
> -       }
>         # Darwin doesn't have VSX, so no soft support for float128.
>         if { [istarget *-*-darwin*] } {
>             return 0
> @@ -7146,12 +7129,6 @@ proc check_effective_target_powerpc_float128_hw_ok
> { } {
>      if { ([istarget powerpc*-*-*]
>           && ![istarget powerpc-*-linux*paired*])
>          || [istarget rs6000-*-*] } {
> -       # AltiVec is not supported on AIX before 5.3.
> -       if { [istarget powerpc*-*-aix4*]
> -            || [istarget powerpc*-*-aix5.1*]
> -            || [istarget powerpc*-*-aix5.2*] } {
> -           return 0
> -       }
>         # Darwin doesn't run on any machine with float128 h/w so far.
>         if { [istarget *-*-darwin*] } {
>             return 0
> @@ -7215,12 +7192,6 @@ proc check_effective_target_powerpc_vsx_ok { } {
>      if { ([istarget powerpc*-*-*]
>           && ![istarget powerpc-*-linux*paired*])
>          || [istarget rs6000-*-*] } {
> -       # VSX is not supported on AIX before 7.1.
> -       if { [istarget powerpc*-*-aix4*]
> -            || [istarget powerpc*-*-aix5*]
> -            || [istarget powerpc*-*-aix6*] } {
> -           return 0
> -       }
>         # Darwin doesn't have VSX, even if it's used with an assembler
>         # which recognises the insns.
>         if { [istarget *-*-darwin*] } {
> --
> 2.39.1
>
diff mbox series

Patch

diff --git a/gcc/testsuite/lib/target-supports.exp b/gcc/testsuite/lib/target-supports.exp
index 3a55b2a4159..16dc2766850 100644
--- a/gcc/testsuite/lib/target-supports.exp
+++ b/gcc/testsuite/lib/target-supports.exp
@@ -6963,11 +6963,6 @@  proc check_effective_target_powerpc_altivec_ok { } {
     # Paired Single, then not ok
     if { [istarget powerpc-*-linux*paired*] } { return 0 }

-    # AltiVec is not supported on AIX before 5.3.
-    if { [istarget powerpc*-*-aix4*]
-	 || [istarget powerpc*-*-aix5.1*]
-	 || [istarget powerpc*-*-aix5.2*] } { return 0 }
-
     # Return true iff compiling with -maltivec does not error.
     return [check_no_compiler_messages powerpc_altivec_ok object {
 	int dummy;
@@ -6980,12 +6975,6 @@  proc check_effective_target_powerpc_p9modulo_ok { } {
     if { ([istarget powerpc*-*-*]
          && ![istarget powerpc-*-linux*paired*])
 	 || [istarget rs6000-*-*] } {
-	# AltiVec is not supported on AIX before 5.3.
-	if { [istarget powerpc*-*-aix4*]
-	     || [istarget powerpc*-*-aix5.1*]
-	     || [istarget powerpc*-*-aix5.2*] } {
-	    return 0
-	}
 	return [check_no_compiler_messages powerpc_p9modulo_ok object {
 	    int main (void) {
 		int i = 5, j = 3, r = -1;
@@ -7116,12 +7105,6 @@  proc check_effective_target_powerpc_float128_sw_ok { } {
     if { ([istarget powerpc*-*-*]
          && ![istarget powerpc-*-linux*paired*])
 	 || [istarget rs6000-*-*] } {
-	# AltiVec is not supported on AIX before 5.3.
-	if { [istarget powerpc*-*-aix4*]
-	     || [istarget powerpc*-*-aix5.1*]
-	     || [istarget powerpc*-*-aix5.2*] } {
-	    return 0
-	}
 	# Darwin doesn't have VSX, so no soft support for float128.
 	if { [istarget *-*-darwin*] } {
 	    return 0
@@ -7146,12 +7129,6 @@  proc check_effective_target_powerpc_float128_hw_ok { } {
     if { ([istarget powerpc*-*-*]
          && ![istarget powerpc-*-linux*paired*])
 	 || [istarget rs6000-*-*] } {
-	# AltiVec is not supported on AIX before 5.3.
-	if { [istarget powerpc*-*-aix4*]
-	     || [istarget powerpc*-*-aix5.1*]
-	     || [istarget powerpc*-*-aix5.2*] } {
-	    return 0
-	}
 	# Darwin doesn't run on any machine with float128 h/w so far.
 	if { [istarget *-*-darwin*] } {
 	    return 0
@@ -7215,12 +7192,6 @@  proc check_effective_target_powerpc_vsx_ok { } {
     if { ([istarget powerpc*-*-*]
          && ![istarget powerpc-*-linux*paired*])
 	 || [istarget rs6000-*-*] } {
-	# VSX is not supported on AIX before 7.1.
-	if { [istarget powerpc*-*-aix4*]
-	     || [istarget powerpc*-*-aix5*]
-	     || [istarget powerpc*-*-aix6*] } {
-	    return 0
-	}
 	# Darwin doesn't have VSX, even if it's used with an assembler
 	# which recognises the insns.
 	if { [istarget *-*-darwin*] } {