From c7f599d0996caab1a5ba2c09cb065e053815418b Mon Sep 17 00:00:00 2001 From: Jan Hubicka Date: Mon, 28 Mar 2005 15:16:14 +0200 Subject: [PATCH] tree-inline.c (estimate_num_insns_1): Use declaration to discover argument types where possible. * tree-inline.c (estimate_num_insns_1): Use declaration to discover argument types where possible. From-SVN: r97132 --- gcc/ChangeLog | 5 +++++ gcc/tree-inline.c | 15 ++++++++++++--- 2 files changed, 17 insertions(+), 3 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index eb471ba5190..33a60b1b93b 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2005-03-28 Jan Hubicka + + * tree-inline.c (estimate_num_insns_1): Use declaration to discover argument + types where possible. + 2005-03-26 Per Bothner Make -f[no-]show-column also control non-cpp diagnostics. diff --git a/gcc/tree-inline.c b/gcc/tree-inline.c index 6f46eed8c36..0b38f08724c 100644 --- a/gcc/tree-inline.c +++ b/gcc/tree-inline.c @@ -1403,9 +1403,18 @@ estimate_num_insns_1 (tree *tp, int *walk_subtrees, void *data) break; } - arg = TREE_OPERAND (x, 1); - for (arg = TREE_OPERAND (x, 1); arg; arg = TREE_CHAIN (arg)) - *count += estimate_move_cost (TREE_TYPE (TREE_VALUE (arg))); + /* Our cost must be kept in sync with cgraph_estimate_size_after_inlining + that does use function declaration to figure out the arguments. */ + if (!decl) + { + for (arg = TREE_OPERAND (x, 1); arg; arg = TREE_CHAIN (arg)) + *count += estimate_move_cost (TREE_TYPE (TREE_VALUE (arg))); + } + else + { + for (arg = DECL_ARGUMENTS (decl); arg; arg = TREE_CHAIN (arg)) + *count += estimate_move_cost (TREE_TYPE (arg)); + } *count += PARAM_VALUE (PARAM_INLINE_CALL_COST); break; -- 2.30.2