diff options
author | meyering <meyering@138bc75d-0d04-0410-961f-82ee72b054a4> | 2011-04-20 18:19:03 +0000 |
---|---|---|
committer | meyering <meyering@138bc75d-0d04-0410-961f-82ee72b054a4> | 2011-04-20 18:19:03 +0000 |
commit | dd045aeeddc65c15840ac25e90f729e2805dfc17 (patch) | |
tree | e56a024bf4863c42aaa3f941f466a849bb9d3b82 /gcc/sese.c | |
parent | ff3aed6233b16f02b26f82b3dac7d604aac42641 (diff) | |
download | gcc-dd045aeeddc65c15840ac25e90f729e2805dfc17.tar.gz |
remove useless if-before-free tests
Change "if (E) free (E);" to "free (E);" everywhere except in the
libgo/, intl/, zlib/ and classpath/ directories.
Also transform equivalent variants like
"if (E != NULL) free (E);" and allow an extra cast on the
argument to free. Otherwise, the tested and freed "E"
expressions must be identical, modulo white space.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@172785 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'gcc/sese.c')
-rw-r--r-- | gcc/sese.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/gcc/sese.c b/gcc/sese.c index bfb0276d3a8..3ce83b9f7c1 100644 --- a/gcc/sese.c +++ b/gcc/sese.c @@ -449,8 +449,7 @@ set_rename (htab_t rename_map, tree old_name, tree expr) if (!slot) return; - if (*slot) - free (*slot); + free (*slot); *slot = new_rename_map_elt (old_name, expr); } @@ -676,8 +675,7 @@ if_region_set_false_region (ifsese if_region, sese region) SESE_EXIT (region) = false_edge; - if (if_region->false_region) - free (if_region->false_region); + free (if_region->false_region); if_region->false_region = region; if (slot) |