Commit be95e2b9 by Martin Jambor Committed by Martin Jambor

ipa-cp.c (ipcp_estimate_growth): Return 0 instead of false.

2008-09-18  Martin Jambor  <mjambor@suse.cz>

	* ipa-cp.c (ipcp_estimate_growth): Return 0 instead of false.

	* ipa-prop.c: Correct comments.
	* ipa-prop.h: Likewise.

From-SVN: r140464
parent e2c9111c
2008-09-18 Martin Jambor <mjambor@suse.cz>
* ipa-cp.c (ipcp_estimate_growth): Return 0 instead of false.
* ipa-prop.c: Correct comments.
* ipa-prop.h: Likewise.
2008-09-18 Jan Hubicka <jh@suse.cz> 2008-09-18 Jan Hubicka <jh@suse.cz>
PR middle-end/37448 PR middle-end/37448
......
...@@ -1058,7 +1058,7 @@ ipcp_estimate_growth (struct cgraph_node *node) ...@@ -1058,7 +1058,7 @@ ipcp_estimate_growth (struct cgraph_node *node)
/* If we will be able to fully replace orignal node, we never increase /* If we will be able to fully replace orignal node, we never increase
program size. */ program size. */
if (!need_original) if (!need_original)
return false; return 0;
info = IPA_NODE_REF (node); info = IPA_NODE_REF (node);
count = ipa_get_param_count (info); count = ipa_get_param_count (info);
......
...@@ -184,6 +184,7 @@ struct ipa_node_params ...@@ -184,6 +184,7 @@ struct ipa_node_params
are or will be shared among various passes. */ are or will be shared among various passes. */
/* Set the number of formal parameters. */ /* Set the number of formal parameters. */
static inline void static inline void
ipa_set_param_count (struct ipa_node_params *info, int count) ipa_set_param_count (struct ipa_node_params *info, int count)
{ {
...@@ -191,6 +192,7 @@ ipa_set_param_count (struct ipa_node_params *info, int count) ...@@ -191,6 +192,7 @@ ipa_set_param_count (struct ipa_node_params *info, int count)
} }
/* Return the number of formal parameters. */ /* Return the number of formal parameters. */
static inline int static inline int
ipa_get_param_count (struct ipa_node_params *info) ipa_get_param_count (struct ipa_node_params *info)
{ {
...@@ -200,6 +202,7 @@ ipa_get_param_count (struct ipa_node_params *info) ...@@ -200,6 +202,7 @@ ipa_get_param_count (struct ipa_node_params *info)
/* Return the declaration of Ith formal parameter of the function corresponding /* Return the declaration of Ith formal parameter of the function corresponding
to INFO. Note there is no setter function as this array is built just once to INFO. Note there is no setter function as this array is built just once
using ipa_initialize_node_params. */ using ipa_initialize_node_params. */
static inline tree static inline tree
ipa_get_param (struct ipa_node_params *info, int i) ipa_get_param (struct ipa_node_params *info, int i)
{ {
...@@ -210,6 +213,7 @@ ipa_get_param (struct ipa_node_params *info, int i) ...@@ -210,6 +213,7 @@ ipa_get_param (struct ipa_node_params *info, int i)
the function associated with INFO. Note that there is no setter method as the function associated with INFO. Note that there is no setter method as
the goal is to set all flags when building the array in the goal is to set all flags when building the array in
ipa_detect_param_modifications. */ ipa_detect_param_modifications. */
static inline bool static inline bool
ipa_is_param_modified (struct ipa_node_params *info, int i) ipa_is_param_modified (struct ipa_node_params *info, int i)
{ {
...@@ -220,6 +224,7 @@ ipa_is_param_modified (struct ipa_node_params *info, int i) ...@@ -220,6 +224,7 @@ ipa_is_param_modified (struct ipa_node_params *info, int i)
function associated with INFO. Note that there is no setter method as the function associated with INFO. Note that there is no setter method as the
goal is to set all flags when building the array in goal is to set all flags when building the array in
ipa_detect_called_params. */ ipa_detect_called_params. */
static inline bool static inline bool
ipa_is_param_called (struct ipa_node_params *info, int i) ipa_is_param_called (struct ipa_node_params *info, int i)
{ {
...@@ -227,6 +232,7 @@ ipa_is_param_called (struct ipa_node_params *info, int i) ...@@ -227,6 +232,7 @@ ipa_is_param_called (struct ipa_node_params *info, int i)
} }
/* Flag this node as having callers with variable number of arguments. */ /* Flag this node as having callers with variable number of arguments. */
static inline void static inline void
ipa_set_called_with_variable_arg (struct ipa_node_params *info) ipa_set_called_with_variable_arg (struct ipa_node_params *info)
{ {
...@@ -234,6 +240,7 @@ ipa_set_called_with_variable_arg (struct ipa_node_params *info) ...@@ -234,6 +240,7 @@ ipa_set_called_with_variable_arg (struct ipa_node_params *info)
} }
/* Have we detected this node was called with variable number of arguments? */ /* Have we detected this node was called with variable number of arguments? */
static inline bool static inline bool
ipa_is_called_with_var_arguments (struct ipa_node_params *info) ipa_is_called_with_var_arguments (struct ipa_node_params *info)
{ {
...@@ -259,6 +266,7 @@ struct ipa_edge_args ...@@ -259,6 +266,7 @@ struct ipa_edge_args
are or will be shared among various passes. */ are or will be shared among various passes. */
/* Set the number of actual arguments. */ /* Set the number of actual arguments. */
static inline void static inline void
ipa_set_cs_argument_count (struct ipa_edge_args *args, int count) ipa_set_cs_argument_count (struct ipa_edge_args *args, int count)
{ {
...@@ -266,6 +274,7 @@ ipa_set_cs_argument_count (struct ipa_edge_args *args, int count) ...@@ -266,6 +274,7 @@ ipa_set_cs_argument_count (struct ipa_edge_args *args, int count)
} }
/* Return the number of actual arguments. */ /* Return the number of actual arguments. */
static inline int static inline int
ipa_get_cs_argument_count (struct ipa_edge_args *args) ipa_get_cs_argument_count (struct ipa_edge_args *args)
{ {
...@@ -275,6 +284,7 @@ ipa_get_cs_argument_count (struct ipa_edge_args *args) ...@@ -275,6 +284,7 @@ ipa_get_cs_argument_count (struct ipa_edge_args *args)
/* Returns a pointer to the jump function for the ith argument. Please note /* Returns a pointer to the jump function for the ith argument. Please note
there is no setter function as jump functions are all set up in there is no setter function as jump functions are all set up in
ipa_compute_jump_functions. */ ipa_compute_jump_functions. */
static inline struct ipa_jump_func * static inline struct ipa_jump_func *
ipa_get_ith_jump_func (struct ipa_edge_args *args, int i) ipa_get_ith_jump_func (struct ipa_edge_args *args, int i)
{ {
...@@ -285,7 +295,7 @@ ipa_get_ith_jump_func (struct ipa_edge_args *args, int i) ...@@ -285,7 +295,7 @@ ipa_get_ith_jump_func (struct ipa_edge_args *args, int i)
typedef struct ipa_node_params ipa_node_params_t; typedef struct ipa_node_params ipa_node_params_t;
typedef struct ipa_edge_args ipa_edge_args_t; typedef struct ipa_edge_args ipa_edge_args_t;
/* Types of vectors hodling the infos. */ /* Types of vectors holding the infos. */
DEF_VEC_O (ipa_node_params_t); DEF_VEC_O (ipa_node_params_t);
DEF_VEC_ALLOC_O (ipa_node_params_t, heap); DEF_VEC_ALLOC_O (ipa_node_params_t, heap);
DEF_VEC_O (ipa_edge_args_t); DEF_VEC_O (ipa_edge_args_t);
...@@ -318,7 +328,8 @@ void free_all_ipa_structures_after_iinln (void); ...@@ -318,7 +328,8 @@ void free_all_ipa_structures_after_iinln (void);
void ipa_register_cgraph_hooks (void); void ipa_register_cgraph_hooks (void);
/* This function ensures the array of node param infos is big enough to /* This function ensures the array of node param infos is big enough to
accomdate a structure for all nodes and realloacates it if not. */ accommodate a structure for all nodes and reallocates it if not. */
static inline void static inline void
ipa_check_create_node_params (void) ipa_check_create_node_params (void)
{ {
...@@ -332,8 +343,9 @@ ipa_check_create_node_params (void) ...@@ -332,8 +343,9 @@ ipa_check_create_node_params (void)
ipa_node_params_vector, cgraph_max_uid + 1); ipa_node_params_vector, cgraph_max_uid + 1);
} }
/* This function ensures the array of adge arguments infos is big enough to /* This function ensures the array of edge arguments infos is big enough to
accomdate a structure for all edges and realloacates it if not. */ accommodate a structure for all edges and reallocates it if not. */
static inline void static inline void
ipa_check_create_edge_args (void) ipa_check_create_edge_args (void)
{ {
...@@ -347,9 +359,10 @@ ipa_check_create_edge_args (void) ...@@ -347,9 +359,10 @@ ipa_check_create_edge_args (void)
cgraph_edge_max_uid + 1); cgraph_edge_max_uid + 1);
} }
/* Returns true if the array of edge infos is large enough to accomodate an /* Returns true if the array of edge infos is large enough to accommodate an
info for EDGE. The main purpose of this function is that debug dumping info for EDGE. The main purpose of this function is that debug dumping
function can check info availability without causing reallocations. */ function can check info availability without causing reallocations. */
static inline bool static inline bool
ipa_edge_args_info_available_for_edge_p (struct cgraph_edge *edge) ipa_edge_args_info_available_for_edge_p (struct cgraph_edge *edge)
{ {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment