summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIlya Enkovich <ilya.enkovich@intel.com>2015-05-13 08:29:14 +0000
committerIlya Enkovich <ienkovich@gcc.gnu.org>2015-05-13 08:29:14 +0000
commit7947dd0bdf9cfe2a704355b1e5d7c7f786c67b9b (patch)
treeb20f108b8c558f2565a2e59340297658b4211432
parentf27127ceddb656b988c21da5f3ef9fa8a7f44afe (diff)
downloadgcc-7947dd0bdf9cfe2a704355b1e5d7c7f786c67b9b.tar.gz
re PR target/65103 ([i386] GOTOFF relocation is not propagated into address expression)
gcc/ PR target/65103 * config/i386/i386.c (ix86_rtx_costs): We want to propagate link time constants into adress expressions and therefore set their cost to 0. gcc/testsuite/ PR target/65103 * gcc.target/i386/pr65103-3.c: New. From-SVN: r223119
-rw-r--r--gcc/ChangeLog7
-rw-r--r--gcc/config/i386/i386.c4
-rw-r--r--gcc/testsuite/ChangeLog5
-rw-r--r--gcc/testsuite/gcc.target/i386/pr65103-3.c19
4 files changed, 34 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 1906845ed58..2bb07a57023 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,10 @@
+2015-05-13 Ilya Enkovich <ilya.enkovich@intel.com>
+
+ PR target/65103
+ * config/i386/i386.c (ix86_rtx_costs): We want to propagate
+ link time constants into adress expressions and therefore set
+ their cost to 0.
+
2015-05-13 Jakub Jelinek <jakub@redhat.com>
PR target/66112
diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index bda32570d54..de333d88a88 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -42006,7 +42006,9 @@ ix86_rtx_costs (rtx x, int code_i, int outer_code_i, int opno, int *total,
&& !(TARGET_64BIT
&& (GET_CODE (x) == LABEL_REF
|| (GET_CODE (x) == SYMBOL_REF
- && SYMBOL_REF_LOCAL_P (x)))))
+ && SYMBOL_REF_LOCAL_P (x))))
+ /* Use 0 cost for CONST to improve its propagation. */
+ && (TARGET_64BIT || GET_CODE (x) != CONST))
*total = 1;
else
*total = 0;
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 0f6b188e15f..b5f96f3693a 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2015-05-13 Ilya Enkovich <ilya.enkovich@intel.com>
+
+ PR target/65103
+ * gcc.target/i386/pr65103-3.c: New.
+
2015-05-13 Martin Liska <mliska@suse.cz>
* g++.dg/ipa/pr65557.C: Remove unnecessary dump flag.
diff --git a/gcc/testsuite/gcc.target/i386/pr65103-3.c b/gcc/testsuite/gcc.target/i386/pr65103-3.c
new file mode 100644
index 00000000000..eddf20bc04b
--- /dev/null
+++ b/gcc/testsuite/gcc.target/i386/pr65103-3.c
@@ -0,0 +1,19 @@
+/* { dg-do compile { target ia32 } } */
+/* { dg-require-effective-target pie } */
+/* { dg-options "-O2 -fPIE" } */
+/* { dg-final { scan-assembler-not "GOTOFF," } } */
+
+static int *data[100];
+
+void test (long a, long b)
+{
+ do
+ {
+ if( a < b )
+ {
+ data[a] = data[b];
+ a++;
+ }
+ }
+ while (a <= b);
+}