This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, PR86257, i386/debug] Fix insn prefix in tls_global_dynamic_64_<mode>
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Rainer Orth <ro at CeBiTec dot Uni-Bielefeld dot DE>
- Cc: Tom de Vries <tdevries at suse dot de>, Jan Hubicka <hubicka at ucw dot cz>, gcc-patches at gcc dot gnu dot org, Uros Bizjak <ubizjak at gmail dot com>
- Date: Tue, 26 Jun 2018 12:37:48 +0200
- Subject: Re: [PATCH, PR86257, i386/debug] Fix insn prefix in tls_global_dynamic_64_<mode>
- References: <20180624115027.zjjyorpdpb32ekm6@localhost.localdomain> <20180624215648.GB66444@kam.mff.cuni.cz> <91150fb0-e08c-3e59-1c47-ff0112fa0b05@suse.de> <yddo9fx7vkv.fsf@CeBiTec.Uni-Bielefeld.DE>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Jun 26, 2018 at 11:20:32AM +0200, Rainer Orth wrote:
> > Committed (after moving the testcase to gcc.target/i386).
>
> the new testcase FAILs on 32-bit Solaris/x86
>
> FAIL: gcc.target/i386/pr86257.c scan-assembler data16[ \\t]*leaq
>
> and, according to gcc-testresults, also on i586-unknown-freebsd10.4 and
> x86_64-pc-linux-gnu.
Yeah, it failed for me with -m32 or -mx32, and also fails with
-mtls-dialect=gnu2. So I've installed following on top of your patch as
obvious.
2018-06-26 Jakub Jelinek <jakub@redhat.com>
PR debug/86257
* gcc.target/i386/pr86257.c: Add -mtls-dialect=gnu to dg-options.
--- gcc/testsuite/gcc.target/i386/pr86257.c.jj 2018-06-25 14:51:20.481986807 +0200
+++ gcc/testsuite/gcc.target/i386/pr86257.c 2018-06-26 12:32:10.284048124 +0200
@@ -1,7 +1,7 @@
/* { dg-require-effective-target lp64 } */
/* { dg-require-effective-target fpic } */
/* { dg-require-effective-target tls } */
-/* { dg-options "-g -fPIC" } */
+/* { dg-options "-g -fPIC -mtls-dialect=gnu" } */
__thread int i;
Jakub