[Bug target/43597] Move and compare with 0 can be combined

2011-08-17 Thread vries at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43597 vries at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED CC|

[Bug target/43597] Move and compare with 0 can be combined

2011-08-17 Thread vries at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43597 --- Comment #8 from vries at gcc dot gnu.org 2011-08-17 11:39:10 UTC --- Author: vries Date: Wed Aug 17 11:39:06 2011 New Revision: 177827 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=177827 Log: 2011-08-17 Tom de Vries PR target/

[Bug target/43597] Move and compare with 0 can be combined

2010-04-22 Thread carrot at google dot com
--- Comment #7 from carrot at google dot com 2010-04-22 12:26 --- (In reply to comment #6) > I can't see how it would hurt to allow combine to always merge insns that are > known to be consecutive (ie to ignore CLASS_LIKELY_SPILLED_P if > prev_nonenote_insn(consumer) == producer). > I

[Bug target/43597] Move and compare with 0 can be combined

2010-04-17 Thread rearnsha at gcc dot gnu dot org
--- Comment #6 from rearnsha at gcc dot gnu dot org 2010-04-17 17:15 --- I can't see how it would hurt to allow combine to always merge insns that are known to be consecutive (ie to ignore CLASS_LIKELY_SPILLED_P if prev_nonenote_insn(consumer) == producer). -- http://gcc.gnu.org/bu

[Bug target/43597] Move and compare with 0 can be combined

2010-03-31 Thread steven at gcc dot gnu dot org
--- Comment #5 from steven at gcc dot gnu dot org 2010-03-31 22:51 --- ...remove accidental CC-list additions... -- steven at gcc dot gnu dot org changed: What|Removed |Added -

[Bug target/43597] Move and compare with 0 can be combined

2010-03-31 Thread steven at gcc dot gnu dot org
--- Comment #4 from steven at gcc dot gnu dot org 2010-03-31 22:49 --- It's not very hard to add a define_peephole2 pattern for this case also, although it's a bit of a hack. I'm not even sure if it would handle the case cmp+mov and mov+cmp case -- does peephole2 care about the order of

[Bug target/43597] Move and compare with 0 can be combined

2010-03-31 Thread steven at gcc dot gnu dot org
--- Comment #3 from steven at gcc dot gnu dot org 2010-03-31 15:15 --- Well, according to GDB, combine does try to combine in try_combine: Breakpoint 8, try_combine (i3=0x204fc8b8, i2=0x204fc828, i1=0x0, new_direct_jump_p=0x6fd4ae28) at ../../trunk/gcc/combine.c:2

[Bug target/43597] Move and compare with 0 can be combined

2010-03-31 Thread steven at gcc dot gnu dot org
--- Comment #2 from steven at gcc dot gnu dot org 2010-03-31 14:59 --- In .final, the insns look like this: @(insn:TI 9 7 8 t.c:8 (set (reg:CC 24 cc) @(compare:CC (reg:SI 0 r0 [orig:134 f ] [134]) @(const_int 0 [0x0]))) 220 {*arm_cmpsi_insn} (expr_list:REG_DEAD (reg:S

[Bug target/43597] Move and compare with 0 can be combined

2010-03-31 Thread ramana at gcc dot gnu dot org
-- ramana at gcc dot gnu dot org changed: What|Removed |Added Severity|normal |enhancement Keywords||missed-opt

[Bug target/43597] Move and compare with 0 can be combined

2010-03-31 Thread steven at gcc dot gnu dot org
-- steven at gcc dot gnu dot org changed: What|Removed |Added Status|UNCONFIRMED |NEW Ever Confirmed|0 |1 Last reconfir

[Bug target/43597] Move and compare with 0 can be combined

2010-03-31 Thread carrot at google dot com
--- Comment #1 from carrot at google dot com 2010-03-31 07:50 --- Created an attachment (id=20262) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20262&action=view) test case -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43597