require tls_runtime for tls execution test
Jakub Jelinek
jakub@redhat.com
Wed Apr 8 15:05:52 GMT 2020
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
More information about the Gcc-patches
mailing list