diff options
author | Jakub Jelinek <jakub@redhat.com> | 2010-10-11 08:17:45 +0200 |
---|---|---|
committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2010-10-11 08:17:45 +0200 |
commit | 922a06c370916f82d58f43a782075994ae7b4009 (patch) | |
tree | 377e2b3bf614a01891d536ccc1e426d22fa853b1 | |
parent | fdcbbfe70c66b9c793f7e0cde99e1acdbc299990 (diff) | |
download | gcc-922a06c370916f82d58f43a782075994ae7b4009.zip gcc-922a06c370916f82d58f43a782075994ae7b4009.tar.gz gcc-922a06c370916f82d58f43a782075994ae7b4009.tar.bz2 |
re PR target/45870 (note: non-delegitimized UNSPEC 5 found (-O1 -g))
PR target/45870
* config/i386/i386.c (ix86_delegitimize_tls_address): New function.
(ix86_delegitimize_address): Use it.
* gcc.dg/tls/pr45870.c: New test.
From-SVN: r165270
-rw-r--r-- | gcc/ChangeLog | 6 | ||||
-rw-r--r-- | gcc/config/i386/i386.c | 49 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/testsuite/gcc.dg/tls/pr45870.c | 21 |
4 files changed, 78 insertions, 3 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index e348c8c..50f8943 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2010-10-11 Jakub Jelinek <jakub@redhat.com> + + PR target/45870 + * config/i386/i386.c (ix86_delegitimize_tls_address): New function. + (ix86_delegitimize_address): Use it. + 2010-10-10 Eric Botcazou <ebotcazou@adacore.com> * opt-functions.awk (opt_sanitized_name): Remove gdwarf+ handling. diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c index 999c716..f6c4eb4 100644 --- a/gcc/config/i386/i386.c +++ b/gcc/config/i386/i386.c @@ -12305,6 +12305,49 @@ ix86_pic_register_p (rtx x) return REG_P (x) && REGNO (x) == PIC_OFFSET_TABLE_REGNUM; } +/* Helper function for ix86_delegitimize_address. + Attempt to delegitimize TLS local-exec accesses. */ + +static rtx +ix86_delegitimize_tls_address (rtx orig_x) +{ + rtx x = orig_x, unspec; + struct ix86_address addr; + + if (!TARGET_TLS_DIRECT_SEG_REFS) + return orig_x; + if (MEM_P (x)) + x = XEXP (x, 0); + if (GET_CODE (x) != PLUS || GET_MODE (x) != Pmode) + return orig_x; + if (ix86_decompose_address (x, &addr) == 0 + || addr.seg != (TARGET_64BIT ? SEG_FS : SEG_GS) + || addr.disp == NULL_RTX + || GET_CODE (addr.disp) != CONST) + return orig_x; + unspec = XEXP (addr.disp, 0); + if (GET_CODE (unspec) == PLUS && CONST_INT_P (XEXP (unspec, 1))) + unspec = XEXP (unspec, 0); + if (GET_CODE (unspec) != UNSPEC || XINT (unspec, 1) != UNSPEC_NTPOFF) + return orig_x; + x = XVECEXP (unspec, 0, 0); + gcc_assert (GET_CODE (x) == SYMBOL_REF); + if (unspec != XEXP (addr.disp, 0)) + x = gen_rtx_PLUS (Pmode, x, XEXP (XEXP (addr.disp, 0), 1)); + if (addr.index) + { + rtx idx = addr.index; + if (addr.scale != 1) + idx = gen_rtx_MULT (Pmode, idx, GEN_INT (addr.scale)); + x = gen_rtx_PLUS (Pmode, idx, x); + } + if (addr.base) + x = gen_rtx_PLUS (Pmode, addr.base, x); + if (MEM_P (orig_x)) + x = replace_equiv_address_nv (orig_x, x); + return x; +} + /* In the name of slightly smaller debug output, and to cater to general assembler lossage, recognize PIC+GOTOFF and turn it back into a direct symbol reference. @@ -12340,7 +12383,7 @@ ix86_delegitimize_address (rtx x) || GET_CODE (XEXP (x, 0)) != UNSPEC || XINT (XEXP (x, 0), 1) != UNSPEC_GOTPCREL || !MEM_P (orig_x)) - return orig_x; + return ix86_delegitimize_tls_address (orig_x); x = XVECEXP (XEXP (x, 0), 0, 0); if (GET_MODE (orig_x) != Pmode) return simplify_gen_subreg (GET_MODE (orig_x), x, Pmode, 0); @@ -12349,7 +12392,7 @@ ix86_delegitimize_address (rtx x) if (GET_CODE (x) != PLUS || GET_CODE (XEXP (x, 1)) != CONST) - return orig_x; + return ix86_delegitimize_tls_address (orig_x); if (ix86_pic_register_p (XEXP (x, 0))) /* %ebx + GOT/GOTOFF */ @@ -12389,7 +12432,7 @@ ix86_delegitimize_address (rtx x) result = XVECEXP (x, 0, 0); if (! result) - return orig_x; + return ix86_delegitimize_tls_address (orig_x); if (const_addend) result = gen_rtx_CONST (Pmode, gen_rtx_PLUS (Pmode, result, const_addend)); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 34783f9..46b7174 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2010-10-11 Jakub Jelinek <jakub@redhat.com> + + PR target/45870 + * gcc.dg/tls/pr45870.c: New test. + 2010-10-10 Janus Weil <janus@gcc.gnu.org> PR fortran/45961 diff --git a/gcc/testsuite/gcc.dg/tls/pr45870.c b/gcc/testsuite/gcc.dg/tls/pr45870.c new file mode 100644 index 0000000..8a5d3c7 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tls/pr45870.c @@ -0,0 +1,21 @@ +/* PR target/45870 */ +/* { dg-do compile } */ +/* { dg-options "-g -O" } */ +/* { dg-require-effective-target tls } */ + +__thread int v[30]; +int bar (void); + +int +foo (int x, int y, int z) +{ + int a, b = z, c; + while (b > 0) + { + c = (bar () % 3); + a = v[x]; + if (x < y) + for (;;); + b += a; + } +} |