[RTL-ifcvt] Reject insns that are multiple_sets

	* ifcvt.c (insn_valid_noce_process_p): Reject insn if it satisfies
	multiple_sets.
	(noce_try_cmove_arith): Add checking asserts that orig_a and orig_b
	are not modified by the final modified insns in the basic blocks.

From-SVN: r231004

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 9278a0c72768e802d224917be62cba1788eb39a7..4510240eb22aae003e65097ca8da7cb182156b38 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,10 @@
+2015-11-27  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
+
+       * ifcvt.c (insn_valid_noce_process_p): Reject insn if it satisfies
+       multiple_sets.
+       (noce_try_cmove_arith): Add checking asserts that orig_a and orig_b
+       are not modified by the final modified insns in the basic blocks.
+
 2015-11-27  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
 
        PR rtl-optimization/68506

diff --git a/gcc/ifcvt.c b/gcc/ifcvt.c
index 8ece8734338c38881254c0fe6f71659f807719d3..c995b0fe5c4810a4bbcd4908975fee963b207659 100644 (file)
--- a/gcc/ifcvt.c
+++ b/gcc/ifcvt.c
@@ -1850,6 +1850,7 @@ insn_valid_noce_process_p (rtx_insn *insn, rtx cc)
 {
   if (!insn
       || !NONJUMP_INSN_P (insn)
+      || multiple_sets (insn)
       || (cc && set_of (cc, insn)))
       return false;
 
@@ -2176,7 +2177,7 @@ noce_try_cmove_arith (struct noce_if_info *if_info)
        swap insn that sets up A with the one that sets up B.  If even
        that doesn't help, punt.  */
 
-  modified_in_a = emit_a != NULL_RTX && modified_in_p (orig_b, emit_a);
+  gcc_checking_assert (!emit_a || !modified_in_p (orig_b, emit_a));
   if (tmp_b && then_bb)
     {
       FOR_BB_INSNS (then_bb, tmp_insn)
@@ -2192,7 +2193,7 @@ noce_try_cmove_arith (struct noce_if_info *if_info)
     }
   if (emit_a || modified_in_a)
     {
-      modified_in_b = emit_b != NULL_RTX && modified_in_p (orig_a, emit_b);
+      gcc_checking_assert (!emit_b || !modified_in_p (orig_a, emit_b));
       if (tmp_b && else_bb)
        {
          FOR_BB_INSNS (else_bb, tmp_insn)