Hi,

Indeed, the split condition needs to be populated. However, I doubt that the 
pattern in question is used by the compiler. Do you have an example where it is 
exercised?

Thanks,
Claudiu
________________________________
From: Kewen Lin <li...@linux.ibm.com>
Sent: Wednesday, June 2, 2021 8:04 AM
To: gcc-patches@gcc.gnu.org <gcc-patches@gcc.gnu.org>
Cc: Kewen Lin <li...@linux.ibm.com>; g...@amylaar.uk <g...@amylaar.uk>; Claudiu 
Zissulescu <claz...@synopsys.com>; andrew.burg...@embecosm.com 
<andrew.burg...@embecosm.com>
Subject: [PATCH 02/11] arc: Update unexpected empty split condition

gcc/ChangeLog:

        * config/arc/arc.md (*bbit_di): Fix empty split condition.
---
 gcc/config/arc/arc.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/config/arc/arc.md b/gcc/config/arc/arc.md
index 7a52551eef5..a03840c4c36 100644
--- a/gcc/config/arc/arc.md
+++ b/gcc/config/arc/arc.md
@@ -5020,7 +5020,7 @@ (define_insn_and_split "*bbit_di"
    (clobber (reg:CC_ZN CC_REG))]
   "!CROSSING_JUMP_P (insn)"
   "#"
-  ""
+  "&& 1"
   [(parallel
      [(set (pc) (if_then_else (match_dup 3) (label_ref (match_dup 0)) (pc)))
       (clobber (reg:CC_ZN CC_REG))])]
--
2.17.1

Reply via email to