This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix some nits in clear_unused_block_pointer
- From: Ian Lance Taylor <iant at google dot com>
- To: Dehao Chen <dehao at google dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Guenther <richard dot guenther at gmail dot com>
- Date: Wed, 26 Sep 2012 14:38:37 -0700
- Subject: Re: [PATCH] Fix some nits in clear_unused_block_pointer
- References: <CAO2gOZXTE7ByVV67CEWKxjr8kg_3CFixB9FQdOb9PUg3Ymgr=w@mail.gmail.com>
On Wed, Sep 26, 2012 at 12:46 PM, Dehao Chen <dehao@google.com> wrote:
> This patch fixes some small nits.
>
> Bootstrapped and passed gcc regression tests.
>
> Is it ok for trunk?
>
> Thanks,
> Dehao
>
> gcc/ChangeLog:
>
> 2012-09-26 Dehao Chen <dehao@google.com>
>
> * tree-ssa-live.c (clear_unused_block_pointer): Fix two small nits.
I know it's tedious, but this is meaningless to a future reader.
Better to say what you changed. E.g.,
* tree-ssa-live.c (clear_unused_block_pointer): Use explicit
(void) for parameters. Don't use LOCATION_BLOCK if it is NULL.
> static void
> -clear_unused_block_pointer ()
> +clear_unused_block_pointer (void)
I don't think we need to say (void) any more now that we use C++. But
I guess it does no harm.
> FOR_EACH_EDGE (e, ei, bb->succs)
> - if (e->goto_locus)
> + if (LOCATION_BLOCK (e->goto_locus))
> TREE_USED (LOCATION_BLOCK (e->goto_locus)) = true;
Please write if (LOCATION_BLOCK (e->goto_locus) != NULL).
This is OK with those changes.
Thanks.
Ian