aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVladimir Makarov <vmakarov@redhat.com>2017-04-08 19:18:42 +0000
committerVladimir Makarov <vmakarov@gcc.gnu.org>2017-04-08 19:18:42 +0000
commit8d2696f95892c43a2caab289143f81ecb5db8cfa (patch)
tree3eaf132d5099b6da1b70003599aaa544d1824013
parent8cfd4a906455b29296c85178b1766aba743a9339 (diff)
downloadgcc-8d2696f95892c43a2caab289143f81ecb5db8cfa.zip
gcc-8d2696f95892c43a2caab289143f81ecb5db8cfa.tar.gz
gcc-8d2696f95892c43a2caab289143f81ecb5db8cfa.tar.bz2
re PR rtl-optimization/70478 ([LRA] S/390: Performance regression - superfluous stack frame)
2017-04-08 Vladimir Makarov <vmakarov@redhat.com> PR rtl-optimization/70478 * lra-constraints.c: Reverse the last patch. From-SVN: r246789
-rw-r--r--gcc/ChangeLog5
-rw-r--r--gcc/lra-constraints.c15
2 files changed, 5 insertions, 15 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 368765e..be2e494 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2017-04-08 Vladimir Makarov <vmakarov@redhat.com>
+
+ PR rtl-optimization/70478
+ * lra-constraints.c: Reverse the last patch.
+
2017-04-08 Andreas Tobler <andreast@gcc.gnu.org>
* config/aarch64/aarch64-freebsd.h: Define MCOUNT_NAME.
diff --git a/gcc/lra-constraints.c b/gcc/lra-constraints.c
index 9440205..4d98624 100644
--- a/gcc/lra-constraints.c
+++ b/gcc/lra-constraints.c
@@ -2685,21 +2685,6 @@ process_alt_operands (int only_alternative)
}
}
- /* When we use memory operand, the insn should read the
- value from memory and even if we just wrote a value
- into the memory it is costly in comparison with an
- insn alternative which does not use memory
- (e.g. register or immediate operand). */
- if (no_regs_p && offmemok)
- {
- if (lra_dump_file != NULL)
- fprintf
- (lra_dump_file,
- " Using memory insn operand %d: reject+=3\n",
- nop);
- reject += 3;
- }
-
#ifdef SECONDARY_MEMORY_NEEDED
/* If reload requires moving value through secondary
memory, it will need one more insn at least. */