Re: [PATCH 05/11] h8300: Update unexpected empty split condition

2021-06-02 Thread Jeff Law via Gcc-patches




On 6/1/2021 11:04 PM, Kewen Lin wrote:

gcc/ChangeLog:

* config/h8300/combiner.md (*andsi3_lshiftrt_n_sb): Fix empty split
condition.
Hold off on this.  We may need a stronger condition in there and that's 
something I'm in the process of cleaning up in the H8 port.





jeff



[PATCH 05/11] h8300: Update unexpected empty split condition

2021-06-01 Thread Kewen Lin via Gcc-patches
gcc/ChangeLog:

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

diff --git a/gcc/config/h8300/combiner.md b/gcc/config/h8300/combiner.md
index 20e19da0419..e31bd507a6f 100644
--- a/gcc/config/h8300/combiner.md
+++ b/gcc/config/h8300/combiner.md
@@ -271,7 +271,7 @@ (define_insn_and_split "*andsi3_lshiftrt_n_sb"
   "exact_log2 (INTVAL (operands[3])) < 16
&& INTVAL (operands[2]) + exact_log2 (INTVAL (operands[3])) == 31"
   "#"
-  ""
+  "&& 1"
   [(parallel [(set (match_dup 0)
   (and:SI (lshiftrt:SI (match_dup 1) (match_dup 2))
   (match_dup 3)))
-- 
2.17.1