This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] Create cfgrtl.h
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Andrew MacLeod <amacleod at redhat dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, Jeff Law <law at redhat dot com>
- Date: Mon, 20 Oct 2014 10:02:10 +0200
- Subject: Re: [patch] Create cfgrtl.h
- Authentication-results: sourceware.org; auth=none
- References: <5441475E dot 1010801 at redhat dot com>
On Fri, Oct 17, 2014 at 6:44 PM, Andrew MacLeod <amacleod@redhat.com> wrote:
> Rather than trying to flatten basic-block.h and do all the work associated
> in one big patch, I'll try to do it in smaller steps :-)
>
> This patch creates cfgrtl.h to maintain the prototypes for functions
> exported from cfgrtl.c. For the moment, basic-block.h includes cfgrtl.h,
> keeping everything compiling.
>
> When basic-block.h gets flattened, I'll reduce inclusion of cfgrtl.h to only
> files which actually need it.
>
> I also took a couple of trivial things out of basic-block.h that didn't
> belong there:
> - extern const struct gcov_ctr_summary *profile_info; belonged in
> profile.h since it is exported from profile.c.. This required a few .c files
> to include profile.h now.
> - the prototypes for gt_ggc_mx (edge_def *e) and gt_pch_nx (edge_def *e)
> were moved to tree-cfg.h since they are delcared in tree-cfg.c.
>
> Bootstraps on x86_64-unknown-linux-gnu, and running test regressions, but
> compilation is likely to be enough to confirm its correct.
>
> Assuming all is fine, OK for trunk?
Ok.
Thanks,
Richard.
> Andrew