diff options
author | Vladimir Makarov <vmakarov@redhat.com> | 2017-04-08 19:18:42 +0000 |
---|---|---|
committer | Vladimir Makarov <vmakarov@gcc.gnu.org> | 2017-04-08 19:18:42 +0000 |
commit | 8d2696f95892c43a2caab289143f81ecb5db8cfa (patch) | |
tree | 3eaf132d5099b6da1b70003599aaa544d1824013 | |
parent | 8cfd4a906455b29296c85178b1766aba743a9339 (diff) | |
download | gcc-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/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/lra-constraints.c | 15 |
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. */ |