diff options
| author | Jeff Law <law@redhat.com> | 2020-02-12 12:12:22 -0700 |
|---|---|---|
| committer | Jeff Law <law@redhat.com> | 2020-02-12 12:12:22 -0700 |
| commit | 37462a131c528d0980915d98567361aa9396b030 (patch) | |
| tree | ceffec490d9d54a933446615ce6208e2b1030ce7 /gcc | |
| parent | 5bfc8303ffe2d86e938d45f13cd99a39469dac4f (diff) | |
| download | gcc-37462a131c528d0980915d98567361aa9396b030.zip gcc-37462a131c528d0980915d98567361aa9396b030.tar.gz gcc-37462a131c528d0980915d98567361aa9396b030.tar.bz2 | |
Drop unused comparison shortening pattern and consolidate remaining comparison shortening patterns.
* config/h8300/h8300.md (comparison shortening peepholes): Drop
(and (xor)) variant. Combine other two into single peephole.
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/config/h8300/h8300.md | 69 |
2 files changed, 12 insertions, 62 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 735cc47..1927290 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2020-02-12 Jeff Law <law@redhat.com> + + * config/h8300/h8300.md (comparison shortening peepholes): Drop + (and (xor)) variant. Combine other two into single peephole. + 2020-02-12 Wilco Dijkstra <wdijkstr@arm.com> PR rtl-optimization/93565 diff --git a/gcc/config/h8300/h8300.md b/gcc/config/h8300/h8300.md index 0548368..dcc9c36 100644 --- a/gcc/config/h8300/h8300.md +++ b/gcc/config/h8300/h8300.md @@ -5405,7 +5405,9 @@ [(cc0) (const_int 0)]) (label_ref (match_operand 2 "" "")) (pc)))] - "peep2_reg_dead_p (2, operands[0])" + "((const_int_qi_operand (operands[1], QImode) + || const_int_hi_operand (operands[1], HImode)) + && peep2_reg_dead_p (2, operands[0]))" [(set (match_dup 4) (and:QI (match_dup 4) (match_dup 5))) @@ -5416,68 +5418,11 @@ (label_ref (match_dup 2)) (pc)))] { - operands[4] = gen_rtx_REG (QImode, REGNO (operands[0])); - operands[5] = gen_int_mode (INTVAL (operands[1]), QImode); - }) + enum machine_mode mode; -(define_peephole2 - [(set (match_operand:SI 0 "register_operand" "") - (and:SI (match_dup 0) - (match_operand:SI 1 "const_int_hi_operand" ""))) - (set (cc0) (compare (match_dup 0) - (const_int 0))) - (set (pc) - (if_then_else (match_operator 3 "eqne_operator" - [(cc0) (const_int 0)]) - (label_ref (match_operand 2 "" "")) - (pc)))] - "peep2_reg_dead_p (2, operands[0])" - [(set (match_dup 4) - (and:HI (match_dup 4) - (match_dup 5))) - (set (cc0) (compare (match_dup 4) - (const_int 0))) - (set (pc) - (if_then_else (match_op_dup 3 [(cc0) (const_int 0)]) - (label_ref (match_dup 2)) - (pc)))] - { - operands[4] = gen_rtx_REG (HImode, REGNO (operands[0])); - operands[5] = gen_int_mode (INTVAL (operands[1]), HImode); - }) - -(define_peephole2 - [(set (match_operand:SI 0 "register_operand" "") - (and:SI (match_dup 0) - (match_operand:SI 1 "const_int_qi_operand" ""))) - (set (match_dup 0) - (xor:SI (match_dup 0) - (match_operand:SI 2 "const_int_qi_operand" ""))) - (set (cc0) (compare (match_dup 0) - (const_int 0))) - (set (pc) - (if_then_else (match_operator 4 "eqne_operator" - [(cc0) (const_int 0)]) - (label_ref (match_operand 3 "" "")) - (pc)))] - "peep2_reg_dead_p (3, operands[0]) - && (~INTVAL (operands[1]) & INTVAL (operands[2])) == 0" - [(set (match_dup 5) - (and:QI (match_dup 5) - (match_dup 6))) - (set (match_dup 5) - (xor:QI (match_dup 5) - (match_dup 7))) - (set (cc0) (compare (match_dup 5) - (const_int 0))) - (set (pc) - (if_then_else (match_op_dup 4 [(cc0) (const_int 0)]) - (label_ref (match_dup 3)) - (pc)))] - { - operands[5] = gen_rtx_REG (QImode, REGNO (operands[0])); - operands[6] = gen_int_mode (INTVAL (operands[1]), QImode); - operands[7] = gen_int_mode (INTVAL (operands[2]), QImode); + mode = const_int_qi_operand (operands[1], QImode) ? QImode : HImode; + operands[4] = gen_rtx_REG (mode, REGNO (operands[0])); + operands[5] = gen_int_mode (INTVAL (operands[1]), mode); }) ;; These triggers right at the end of allocation of locals in the |