re PR target/40971 (ICE in memory_address)
PR target/40971 * config/rs6000/rs6000.c (rs6000_legitimize_address): For [DT][FDI]mode ensure the offset isn't 4/8/12 bytes below 0x8000. * gcc.dg/pr40971.c: New test. From-SVN: r150870
This commit is contained in:
parent
9cfa50655c
commit
f5e05b9552
|
@ -1,3 +1,9 @@
|
|||
2009-08-18 Jakub Jelinek <jakub@redhat.com>
|
||||
|
||||
PR target/40971
|
||||
* config/rs6000/rs6000.c (rs6000_legitimize_address): For
|
||||
[DT][FDI]mode ensure the offset isn't 4/8/12 bytes below 0x8000.
|
||||
|
||||
2009-08-16 Uros Bizjak <ubizjak@gmail.com>
|
||||
|
||||
Backport from mainline:
|
||||
|
|
|
@ -3808,6 +3808,8 @@ rtx
|
|||
rs6000_legitimize_address (rtx x, rtx oldx ATTRIBUTE_UNUSED,
|
||||
enum machine_mode mode)
|
||||
{
|
||||
unsigned int extra = 0;
|
||||
|
||||
if (GET_CODE (x) == SYMBOL_REF)
|
||||
{
|
||||
enum tls_model model = SYMBOL_REF_TLS_MODEL (x);
|
||||
|
@ -3815,10 +3817,32 @@ rs6000_legitimize_address (rtx x, rtx oldx ATTRIBUTE_UNUSED,
|
|||
return rs6000_legitimize_tls_address (x, model);
|
||||
}
|
||||
|
||||
switch (mode)
|
||||
{
|
||||
case DFmode:
|
||||
case DDmode:
|
||||
extra = 4;
|
||||
break;
|
||||
case DImode:
|
||||
if (!TARGET_POWERPC64)
|
||||
extra = 4;
|
||||
break;
|
||||
case TFmode:
|
||||
case TDmode:
|
||||
extra = 12;
|
||||
break;
|
||||
case TImode:
|
||||
extra = TARGET_POWERPC64 ? 8 : 12;
|
||||
break;
|
||||
default:
|
||||
break;
|
||||
}
|
||||
|
||||
if (GET_CODE (x) == PLUS
|
||||
&& GET_CODE (XEXP (x, 0)) == REG
|
||||
&& GET_CODE (XEXP (x, 1)) == CONST_INT
|
||||
&& (unsigned HOST_WIDE_INT) (INTVAL (XEXP (x, 1)) + 0x8000) >= 0x10000
|
||||
&& ((unsigned HOST_WIDE_INT) (INTVAL (XEXP (x, 1)) + 0x8000)
|
||||
>= 0x10000 - extra)
|
||||
&& !((TARGET_POWERPC64
|
||||
&& (mode == DImode || mode == TImode)
|
||||
&& (INTVAL (XEXP (x, 1)) & 3) != 0)
|
||||
|
@ -3831,10 +3855,12 @@ rs6000_legitimize_address (rtx x, rtx oldx ATTRIBUTE_UNUSED,
|
|||
HOST_WIDE_INT high_int, low_int;
|
||||
rtx sum;
|
||||
low_int = ((INTVAL (XEXP (x, 1)) & 0xffff) ^ 0x8000) - 0x8000;
|
||||
if (low_int >= 0x8000 - extra)
|
||||
low_int = 0;
|
||||
high_int = INTVAL (XEXP (x, 1)) - low_int;
|
||||
sum = force_operand (gen_rtx_PLUS (Pmode, XEXP (x, 0),
|
||||
GEN_INT (high_int)), 0);
|
||||
return gen_rtx_PLUS (Pmode, sum, GEN_INT (low_int));
|
||||
return plus_constant (sum, low_int);
|
||||
}
|
||||
else if (GET_CODE (x) == PLUS
|
||||
&& GET_CODE (XEXP (x, 0)) == REG
|
||||
|
|
|
@ -1,3 +1,8 @@
|
|||
2009-08-18 Jakub Jelinek <jakub@redhat.com>
|
||||
|
||||
PR target/40971
|
||||
* gcc.dg/pr40971.c: New test.
|
||||
|
||||
2009-08-17 Dodji Seketeli <dodji@redhat.com>
|
||||
|
||||
* gcc.dg/debug/dwarf2/inline2.c: Add -dA to compile options.
|
||||
|
|
|
@ -0,0 +1,23 @@
|
|||
/* PR target/40971 */
|
||||
/* { dg-do compile } */
|
||||
/* { dg-options "-O -fstack-protector -fno-strict-aliasing" } */
|
||||
/* { dg-require-effective-target fstack_protector } */
|
||||
|
||||
extern void bar (char *);
|
||||
|
||||
void
|
||||
foo (int f, long a)
|
||||
{
|
||||
{
|
||||
char d[32768];
|
||||
bar (d);
|
||||
}
|
||||
double b = f;
|
||||
while (a)
|
||||
{
|
||||
char c[sizeof (double)];
|
||||
__builtin_memcpy (c, &b, sizeof (c));
|
||||
if (*(double *) c != 2.0)
|
||||
break;
|
||||
}
|
||||
}
|
Loading…
Reference in New Issue