This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] Finish cleaning up gimple.h
- From: Jeff Law <law at redhat dot com>
- To: Andrew MacLeod <amacleod at redhat dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Cc: Richard Biener <richard dot guenther at gmail dot com>, Diego Novillo <dnovillo at google dot com>
- Date: Fri, 15 Nov 2013 11:32:06 -0700
- Subject: Re: [patch] Finish cleaning up gimple.h
- Authentication-results: sourceware.org; auth=none
- References: <52865033 dot 1060305 at redhat dot com>
On 11/15/13 09:47, Andrew MacLeod wrote:
This patch finally gets gimple.h into its proper state of only having
prototypes for gimple.c.
The prototype for get_base_address was in tree.h, and it seemed more
appropriate there since gimple.h is for gimple statements. I moved the
function to tree.c
2 new .h files were created, cfgexpand.h and tree-nested.h. I flushed
both of those out to include all the exports for the files, so the
'build_addr' prototype was moved from tree.h and
'estimated_stack_frame_size' was moved from tree-inline.h. I looked at
shuffling the exports to other files, but nothing jumped at me.
estimated_stack_frame_size depends on too much other stuff in cfgexpand
to move even though it only has one client.
build_addr may belong somewhere else, but Im not sure where... tree.c as
well maybe? The fortran FE uses it as well...
A few #includes needed to be added to .c files for the moved prototypes.
I left the macro definitions for SCALE, LABEL and PERCENT at the bottom
of gimple.h. I was going to move them, but really have no idea where to
put them. I thought about coretypes.h but that doesn't seem right
either. suggestions?
I think PERCENT is only used by tree-into-ssa.c, so shove it in there.
SCALE and LABEL are only used by tree-cfg.c and tree-dfa.c, if that
helps you narrow down a potential location.
Bootstraps on x86_64-unknown-linux-gnu and regressions are currently
running. Assuming no issues, OK?
Yes. Thanks,
jeff