re PR rtl-optimization/70478 ([LRA] S/390: Performance regression - superfluous stack...

2017-04-08  Vladimir Makarov  <vmakarov@redhat.com>

	PR rtl-optimization/70478
	* lra-constraints.c: Reverse the last patch.

From-SVN: r246789

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 368765ef02d5ac7010b18ba2e3c666d153955349..be2e4945638cee7c253009480dfad1c661c91f5c 100644 (file)
--- 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 9440205b66b5c4633fcf0ec4afea8c6258f5c238..4d986249a6fa6df1ad497d0830395dc3fed23f92 100644 (file)
--- 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.  */