Your inlining change also breaks bootstrap

Jan Hubicka jh@suse.cz
Wed Jul 9 14:30:00 GMT 2003


> 
> This change breaks the Objective-C bootstrap; libobjc does not build
> because the Objective-C front end does not define the new hook, and
> the code uses the hook unconditionally.  In general, new hooks of this
> kind should have a default, fall-back version that can be used.

Oops, I did fix this yesterday before commiting the patch and then
forgot to update the changelog and thus didn't listed the file in commit
command.  I am running bootstrap again and adding the hook.
About the default value, I am not quite sure what to do here.  In order
to esitmate size one must understand the nodes.
For objc I intended to use version from c-common.c.  Do you think it is
good idea to create now dead version in tree-inline.c that simply counts
number of tree nodes?

Honza
> 
> Wed Jul  9 02:25:39 CEST 2003  Jan Hubicka  <jh@suse.cz>
> 
> 	* c-common.c (c_estimate_num_insns_1): New static function.
> 	(c_estimate_num_insns): New global function.
> 	* c-common.h (DECL_NUM_STMTS): Rename to...
> 	(DECL_ESTIMATED_INSNS): ... this.
> 	(c_estimate_num_insns): Declare.
> 	* c-decl.c (duplicate_decls): Use DECL_ESTIMATED_INSNS.
> 	* c-lang.c (LANG_HOOKS_TREE_INLINING_ESTIMATE_NUM_INSNS): New.
> 	* c-semantics.c (add_stmt): Do not account statements.
> 	* langhooks-def.h (LANG_HOOKS_TREE_INLINING_ESTIMATE_NUM_INSNS):
> 	New.
> 
> --
> Mark Mitchell
> CodeSourcery, LLC
> mark@codesourcery.com



More information about the Gcc mailing list