diff options
| author | Mark Mitchell <mark@codesourcery.com> | 2001-06-09 23:08:05 +0000 |
|---|---|---|
| committer | Mark Mitchell <mmitchel@gcc.gnu.org> | 2001-06-09 23:08:05 +0000 |
| commit | e69decfd36cabe29f106ff1c277706ac621e690f (patch) | |
| tree | f0c3c1d31037575d513cf3e9d55b3b78b385ae09 | |
| parent | 3bce8a01efc92473c193976352a6c8c7dd3d1f4a (diff) | |
| download | gcc-e69decfd36cabe29f106ff1c277706ac621e690f.zip gcc-e69decfd36cabe29f106ff1c277706ac621e690f.tar.gz gcc-e69decfd36cabe29f106ff1c277706ac621e690f.tar.bz2 | |
expr.c (expand_expr, [...]): Do not replace an array element with the known initializing value if...
* expr.c (expand_expr, case ARRAY_REF): Do not replace
an array element with the known initializing value if it has
side-effects.
* toplev.c (rest_of_compilation): Remove dead code before
purge_addressof.
From-SVN: r43120
| -rw-r--r-- | gcc/ChangeLog | 9 | ||||
| -rw-r--r-- | gcc/expr.c | 2 | ||||
| -rw-r--r-- | gcc/toplev.c | 10 |
3 files changed, 20 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index b21874bd..3c0d4a5 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,12 @@ +2001-06-09 Mark Mitchell <mark@codesourcery.com> + + * expr.c (expand_expr, case ARRAY_REF): Do not replace + an array element with the known initializing value if it has + side-effects. + + * toplev.c (rest_of_compilation): Remove dead code before + purge_addressof. + 2001-06-09 Neil Booth <neil@daikokuya.demon.co.uk> * cppinit.c (print_help): Document -fpreprocessed. @@ -6891,7 +6891,7 @@ expand_expr (exp, target, tmode, modifier) elem = TREE_CHAIN (elem)) ; - if (elem) + if (elem && !TREE_SIDE_EFFECTS (elem)) return expand_expr (fold (TREE_VALUE (elem)), target, tmode, ro_modifier); } diff --git a/gcc/toplev.c b/gcc/toplev.c index 0e197cc..2b75aa2 100644 --- a/gcc/toplev.c +++ b/gcc/toplev.c @@ -3031,6 +3031,16 @@ rest_of_compilation (decl) timevar_pop (TV_JUMP); } + /* It's important to remove dead code before we call + purge_addressof. Sometimes, the only ADDRESSOFs for a REG + will be dead, and if we don't get rid of them, we will end up + committing ourselves to dumping the REG to the stack + unnecessarily. */ + find_basic_blocks (insns, max_reg_num (), rtl_dump_file); + cleanup_cfg (); + life_analysis (insns, rtl_dump_file, + PROP_SCAN_DEAD_CODE | PROP_KILL_DEAD_CODE); + /* The second pass of jump optimization is likely to have removed a bunch more instructions. */ renumber_insns (rtl_dump_file); |