re PR rtl-optimization/79541 (lra reads uninitialized memory (with invalid input))

2017-02-17  Vladimir Makarov  <vmakarov@redhat.com>

	PR rtl-optimization/79541
	* lra-constraints.c (curr_insn_transform): Remove wrong asm insn
	instead of transforming it into USE.

From-SVN: r245536

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 0339ac9c5d3d66aec401760d85b3a80fcda33e2f..d4a0c0def3555e92fac9c6201ad6d827e0bdbbdb 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2017-02-17  Vladimir Makarov  <vmakarov@redhat.com>
+
+       PR rtl-optimization/79541
+       * lra-constraints.c (curr_insn_transform): Remove wrong asm insn
+       instead of transforming it into USE.
+
 2017-02-17  Segher Boessenkool  <segher@kernel.crashing.org>
 
        * config/rs6000/rs6000.md (extendsfdf2): Remove default arguments.

diff --git a/gcc/lra-constraints.c b/gcc/lra-constraints.c
index 585ca841aee38891ac7861bbb157465040cc40e4..0098a7585e8e40cbd108291f2c60b81497b59a41 100644 (file)
--- a/gcc/lra-constraints.c
+++ b/gcc/lra-constraints.c
@@ -3773,9 +3773,9 @@ curr_insn_transform (bool check_only_p)
        fatal_insn ("unable to generate reloads for:", curr_insn);
       error_for_asm (curr_insn,
                     "inconsistent operand constraints in an %<asm%>");
-      /* Avoid further trouble with this insn. */
-      PATTERN (curr_insn) = gen_rtx_USE (VOIDmode, const0_rtx);
-      lra_invalidate_insn_data (curr_insn);
+      /* Avoid further trouble with this insn.  Don't generate use
+        pattern here as we could use the insn SP offset.  */
+      lra_set_insn_deleted (curr_insn);
       return true;
     }