diff mbox series

require tls_runtime for tls execution test

Message ID orftdednnz.fsf@livre.home
State New
Headers show
Series require tls_runtime for tls execution test | expand

Commit Message

Alexandre Oliva April 8, 2020, 3:04 p.m. UTC
All TLS execution tests require tls_runtime, not just tls; pr78796.c
is the only exception that is not otherwise limited to platforms known
to support it.  I suppose that's an oversight.  On a platform whose
linker is configured to disregard TLS relocations, this test compiles
and assembles successfully, but execution fails.  The tls_runtime
requirement target avoids the noise from the expected failure.

Tested on the affected platform, and on x86_64-linux-gnu.  Ok to install?


for  gcc/testsuite/ChangeLog

	* gcc.dg/tls/pr78796.c: Require tls_runtime.
---
 gcc/testsuite/gcc.dg/tls/pr78796.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Li, Pan2 via Gcc-patches April 8, 2020, 3:05 p.m. UTC | #1
On Wed, Apr 08, 2020 at 12:04:16PM -0300, Alexandre Oliva wrote:
> 
> All TLS execution tests require tls_runtime, not just tls; pr78796.c
> is the only exception that is not otherwise limited to platforms known
> to support it.  I suppose that's an oversight.  On a platform whose
> linker is configured to disregard TLS relocations, this test compiles
> and assembles successfully, but execution fails.  The tls_runtime
> requirement target avoids the noise from the expected failure.
> 
> Tested on the affected platform, and on x86_64-linux-gnu.  Ok to install?
> 
> 
> for  gcc/testsuite/ChangeLog
> 
> 	* gcc.dg/tls/pr78796.c: Require tls_runtime.

Ok.

> ---
>  gcc/testsuite/gcc.dg/tls/pr78796.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/gcc/testsuite/gcc.dg/tls/pr78796.c b/gcc/testsuite/gcc.dg/tls/pr78796.c
> index a0b23d6..038e536 100644
> --- a/gcc/testsuite/gcc.dg/tls/pr78796.c
> +++ b/gcc/testsuite/gcc.dg/tls/pr78796.c
> @@ -2,7 +2,7 @@
>  /* { dg-do run } */
>  /* { dg-options "-O2" } */
>  /* { dg-additional-options "-mcmodel=large" { target aarch64-*-* } } */
> -/* { dg-require-effective-target tls } */
> +/* { dg-require-effective-target tls_runtime } */
>  /* { dg-add-options tls } */
>  
>  struct S { int a, b, c, d, e; };

	Jakub
diff mbox series

Patch

diff --git a/gcc/testsuite/gcc.dg/tls/pr78796.c b/gcc/testsuite/gcc.dg/tls/pr78796.c
index a0b23d6..038e536 100644
--- a/gcc/testsuite/gcc.dg/tls/pr78796.c
+++ b/gcc/testsuite/gcc.dg/tls/pr78796.c
@@ -2,7 +2,7 @@ 
 /* { dg-do run } */
 /* { dg-options "-O2" } */
 /* { dg-additional-options "-mcmodel=large" { target aarch64-*-* } } */
-/* { dg-require-effective-target tls } */
+/* { dg-require-effective-target tls_runtime } */
 /* { dg-add-options tls } */
 
 struct S { int a, b, c, d, e; };