rs6000: Add target supports for "future" system

Message ID 13012539-5f8a-312b-715c-aa8d3fd46576@linux.ibm.com
State New
Headers show
Series
  • rs6000: Add target supports for "future" system
Related show

Commit Message

Bill Schmidt May 30, 2019, 3:52 a.m.
Hi,

This patch from Mike Meissner adds procs to target-supports.exp to support -mcpu=future.

Tested in conjunction with the next patch (test cases) with no problems.
Is this okay for trunk?

Thanks,
Bill


2019-05-29  Michael Meissner  <meissner@linux.ibm.com>

	* lib/target-supports.exp (check_powerpc_future_hw_available):
	New.
	(check_effective_target_powerpc_future_ok): New.

Comments

Segher Boessenkool May 30, 2019, 7:26 p.m. | #1
On Wed, May 29, 2019 at 10:52:38PM -0500, Bill Schmidt wrote:
> 	* lib/target-supports.exp (check_powerpc_future_hw_available):

> 	New.


That fits on one line just fine?

> +# Return 1 if this is a PowerPC target supporting -mfuture.

> +# Limit this to 64-bit linux systems for now until other

> +# targets support FUTURE.

> +

> +proc check_effective_target_powerpc_future_ok { } {

> +    if { ([istarget powerpc64*-*-linux*]) } {


It should be limited to powerpc64le-*-linux, or at least to ELFv2, right?

Not that that should matter at all: you pass -mfuture, and the compiler
will not allow that on unsupported configurations anyway (right? :-) )

But, okay for trunk.  Thanks!


Segher

Patch

diff --git a/gcc/testsuite/lib/target-supports.exp b/gcc/testsuite/lib/target-supports.exp
index 3bd6e815715..982ec8ccc09 100644
--- a/gcc/testsuite/lib/target-supports.exp
+++ b/gcc/testsuite/lib/target-supports.exp
@@ -2140,6 +2140,25 @@  proc check_p9modulo_hw_available { } {
     }]
 }
 
+
+# Return 1 if the target supports executing FUTURE instructions, 0 otherwise.
+# Cache the result.  It is assumed that if a simulator does not support the
+# FUTURE instructions, that it will generate an error and this test will fail.
+
+proc check_powerpc_future_hw_available { } {
+    return [check_cached_effective_target powerpc_future_hw_available {
+	check_runtime_nocache powerpc_future_hw_available {
+	    int main()
+	    {
+		/* Set e first and use +r to check if pli actually works.  */
+		long e = -1;
+		asm ("pli %0,%1" : "+r" (e) : "n" (0x12345));
+		return (e == 0x12345);
+	    }
+	} "-mfuture"
+    }]
+}
+
 # Return 1 if the target supports executing __float128 on PowerPC via software
 # emulation, 0 otherwise.  Cache the result.
 
@@ -5123,6 +5142,24 @@  proc check_effective_target_powerpc_p9modulo_ok { } {
     }
 }
 
+# Return 1 if this is a PowerPC target supporting -mfuture.
+# Limit this to 64-bit linux systems for now until other
+# targets support FUTURE.
+
+proc check_effective_target_powerpc_future_ok { } {
+    if { ([istarget powerpc64*-*-linux*]) } {
+	return [check_no_compiler_messages powerpc_future_ok object {
+	    int main (void) {
+		long e;
+		asm ("pli %0,%1" : "=r" (e) : "n" (0x12345));
+		return e;
+	    }
+	} "-mfuture"]
+    } else {
+	return 0
+    }
+}
+
 # Return 1 if this is a PowerPC target supporting -mfloat128 via either
 # software emulation on power7/power8 systems or hardware support on power9.