diff options
| author | Richard Earnshaw <rearnsha@arm.com> | 2003-06-06 17:19:06 +0000 |
|---|---|---|
| committer | Richard Earnshaw <rearnsha@gcc.gnu.org> | 2003-06-06 17:19:06 +0000 |
| commit | 3a11ec8b5344520f07a639c4bd676e1124c919cc (patch) | |
| tree | 96392c22e7f38eb3f9bb7ab19a5aeb2aa01fd0d8 /gcc/ifcvt.c | |
| parent | 7dc8b126ce1892db6c4380dff070c664ed3e225f (diff) | |
| download | gcc-3a11ec8b5344520f07a639c4bd676e1124c919cc.tar.gz | |
re PR target/11052 ([arm] noce_process_if_block() can loose REG_INC notes)
PR target/11052
* ifcvt.c (noce_process_if_block): Fail if the destination has
side-effects.
gcc.c-torture/execute/20030606-1.c: New.
From-SVN: r67558
Diffstat (limited to 'gcc/ifcvt.c')
| -rw-r--r-- | gcc/ifcvt.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/gcc/ifcvt.c b/gcc/ifcvt.c index baac0413176..87479060784 100644 --- a/gcc/ifcvt.c +++ b/gcc/ifcvt.c @@ -1821,6 +1821,15 @@ noce_process_if_block (ce_info) || modified_between_p (x, PREV_INSN (if_info.cond_earliest), jump)) insn_b = set_b = NULL_RTX; } + + /* If x has side effects then only the if-then-else form is safe to + convert. But even in that case we would need to restore any notes + (such as REG_INC) at then end. That can be tricky if + noce_emit_move_insn expands to more than one insn, so disable the + optimization entirely for now if there are side effects. */ + if (side_effects_p (x)) + return FALSE; + b = (set_b ? SET_SRC (set_b) : x); /* Only operate on register destinations, and even then avoid extending |