]> gcc.gnu.org Git - gcc.git/commitdiff
cgraph.c (cgraph_create_edge_1): Initialize call_stmt_cannot_inline_p from the stmt...
authorRichard Guenther <rguenther@suse.de>
Mon, 5 Dec 2011 08:51:53 +0000 (08:51 +0000)
committerRichard Biener <rguenth@gcc.gnu.org>
Mon, 5 Dec 2011 08:51:53 +0000 (08:51 +0000)
2011-12-05  Richard Guenther  <rguenther@suse.de>

* cgraph.c (cgraph_create_edge_1): Initialize
call_stmt_cannot_inline_p from the stmt if possible.
(cgraph_make_edge_direct): Likewise.
* gimple-streamer-in.c (input_gimple_stmt): Do not
call gimple_call_set_cannot_inline.
* gimple.h (enum gf_mask): Remove GF_CALL_CANNOT_INLINE, shift
values.
(gimple_call_set_cannot_inline): Remove.
(gimple_call_cannot_inline_p): Likewise.
* ipa-inline-analysis.c (initialize_inline_failed): Look
at the edge call_stmt_cannot_inline_p flag.
* ipa-inline.c (can_inline_edge_p): Likewise.
(early_inliner): Only update the edge flag.
* ipa-prop.c (update_indirect_edges_after_inlining): Likewise.
(ipa_modify_call_arguments): Do not call gimple_call_set_cannot_inline.
* cgraphunit.c (assemble_thunk): Likewise.
* gimple-fold.c (gimple_fold_call): Likewise.

* tree.h (CALL_CANNOT_INLINE_P): Remove.
* tree-mudflap.c (mf_xform_statements): Do not modify alloca calls.
* builtins.c (expand_builtin_alloca): With -fmudflap do not expand
alloca calls inline.
* cfgexpand.c (expand_call_stmt): Do not set CALL_CANNOT_INLINE_P.
* gimple.c (gimple_build_call_from_tree): Do not read
CALL_CANNOT_INLINE_P.
* gimplify.c (gimplify_call_expr): Do not copy CALL_CANNOT_INLINE_P.

From-SVN: r182001

15 files changed:
gcc/ChangeLog
gcc/builtins.c
gcc/cfgexpand.c
gcc/cgraph.c
gcc/cgraphunit.c
gcc/gimple-fold.c
gcc/gimple-streamer-in.c
gcc/gimple.c
gcc/gimple.h
gcc/gimplify.c
gcc/ipa-inline-analysis.c
gcc/ipa-inline.c
gcc/ipa-prop.c
gcc/tree-mudflap.c
gcc/tree.h

index 296838111f8140ee3d51910a411b32d06f02de90..ee84c7e7ae4f581e84ee7cdeed9d9c9b5347b5b8 100644 (file)
@@ -1,3 +1,32 @@
+2011-12-05  Richard Guenther  <rguenther@suse.de>
+
+       * cgraph.c (cgraph_create_edge_1): Initialize
+       call_stmt_cannot_inline_p from the stmt if possible.
+       (cgraph_make_edge_direct): Likewise.
+       * gimple-streamer-in.c (input_gimple_stmt): Do not
+       call gimple_call_set_cannot_inline.
+       * gimple.h (enum gf_mask): Remove GF_CALL_CANNOT_INLINE, shift
+       values.
+       (gimple_call_set_cannot_inline): Remove.
+       (gimple_call_cannot_inline_p): Likewise.
+       * ipa-inline-analysis.c (initialize_inline_failed): Look
+       at the edge call_stmt_cannot_inline_p flag.
+       * ipa-inline.c (can_inline_edge_p): Likewise.
+       (early_inliner): Only update the edge flag.
+       * ipa-prop.c (update_indirect_edges_after_inlining): Likewise.
+       (ipa_modify_call_arguments): Do not call gimple_call_set_cannot_inline.
+       * cgraphunit.c (assemble_thunk): Likewise.
+       * gimple-fold.c (gimple_fold_call): Likewise.
+
+       * tree.h (CALL_CANNOT_INLINE_P): Remove.
+       * tree-mudflap.c (mf_xform_statements): Do not modify alloca calls.
+       * builtins.c (expand_builtin_alloca): With -fmudflap do not expand
+       alloca calls inline.
+       * cfgexpand.c (expand_call_stmt): Do not set CALL_CANNOT_INLINE_P.
+       * gimple.c (gimple_build_call_from_tree): Do not read
+       CALL_CANNOT_INLINE_P.
+       * gimplify.c (gimplify_call_expr): Do not copy CALL_CANNOT_INLINE_P.
+
 2011-12-05  Jakub Jelinek  <jakub@redhat.com>
            Eric Botcazou  <ebotcazou@adacore.com>
 
index 5ad69ceca92e36421f3888b9fd2d98efaa9298d8..019da159dd4b9dd3f633404231ec029ffd5538b0 100644 (file)
@@ -4523,8 +4523,8 @@ expand_builtin_alloca (tree exp, bool cannot_accumulate)
   bool alloca_with_align = (DECL_FUNCTION_CODE (get_callee_fndecl (exp))
                            == BUILT_IN_ALLOCA_WITH_ALIGN);
 
-  /* Emit normal call if marked not-inlineable.  */
-  if (CALL_CANNOT_INLINE_P (exp))
+  /* Emit normal call if we use mudflap.  */
+  if (flag_mudflap)
     return NULL_RTX;
 
   valid_arglist
index 15e8231ca722f93a0d34c609fc62de28407c9279..86847217322271eeeb678c9315da5c494ff8c802 100644 (file)
@@ -2050,7 +2050,6 @@ expand_call_stmt (gimple stmt)
     CALL_ALLOCA_FOR_VAR_P (exp) = gimple_call_alloca_for_var_p (stmt);
   else
     CALL_FROM_THUNK_P (exp) = gimple_call_from_thunk_p (stmt);
-  CALL_CANNOT_INLINE_P (exp) = gimple_call_cannot_inline_p (stmt);
   CALL_EXPR_VA_ARG_PACK (exp) = gimple_call_va_arg_pack_p (stmt);
   SET_EXPR_LOCATION (exp, gimple_location (stmt));
   TREE_BLOCK (exp) = gimple_block (stmt);
index ac516abefba96dbbd5f3251f4275dbb03d978ee4..e65ddef8aec79231ca5c71c1c9391fe04ecc3cba 100644 (file)
@@ -988,8 +988,12 @@ cgraph_create_edge_1 (struct cgraph_node *caller, struct cgraph_node *callee,
   edge->can_throw_external
     = call_stmt ? stmt_can_throw_external (call_stmt) : false;
   pop_cfun ();
-  edge->call_stmt_cannot_inline_p =
-    (call_stmt ? gimple_call_cannot_inline_p (call_stmt) : false);
+  if (call_stmt
+      && callee && callee->decl
+      && !gimple_check_call_matching_types (call_stmt, callee->decl))
+    edge->call_stmt_cannot_inline_p = true;
+  else
+    edge->call_stmt_cannot_inline_p = false;
   if (call_stmt && caller->call_site_hash)
     cgraph_add_edge_to_call_site_hash (edge);
 
@@ -1184,12 +1188,9 @@ cgraph_make_edge_direct (struct cgraph_edge *edge, struct cgraph_node *callee)
   /* Insert to callers list of the new callee.  */
   cgraph_set_edge_callee (edge, callee);
 
-  if (edge->call_stmt
-      && !gimple_check_call_matching_types (edge->call_stmt, callee->decl))
-    {
-      gimple_call_set_cannot_inline (edge->call_stmt, true);
-      edge->call_stmt_cannot_inline_p = true;
-    }
+  if (edge->call_stmt)
+    edge->call_stmt_cannot_inline_p
+      = !gimple_check_call_matching_types (edge->call_stmt, callee->decl);
 
   /* We need to re-determine the inlining status of the edge.  */
   initialize_inline_failed (edge);
index 251b55547259accbeed793b932cb2a36bf472c87..886a2def0e56b408548d1f97a0d3e8693207b2a5 100644 (file)
@@ -1694,7 +1694,6 @@ assemble_thunk (struct cgraph_node *node)
         VEC_quick_push (tree, vargs, arg);
       call = gimple_build_call_vec (build_fold_addr_expr_loc (0, alias), vargs);
       VEC_free (tree, heap, vargs);
-      gimple_call_set_cannot_inline (call, true);
       gimple_call_set_from_thunk (call, true);
       if (restmp)
         gimple_call_set_lhs (call, restmp);
index 0da5eef18a2d07218e11c04fc498b4ce8bd70e82..3297f11364b9de3bcfb2cd007b761e2764127dd9 100644 (file)
@@ -1108,23 +1108,12 @@ gimple_fold_call (gimple_stmt_iterator *gsi, bool inplace)
        }
     }
 
-  /* Check whether propagating into the function address made the
-     call direct, and thus possibly non-inlineable.
-     ???  This asks for a more conservative setting of the non-inlinable
-     flag, namely true for all indirect calls.  But that would require
-     that we can re-compute the flag conservatively, thus it isn't
-     ever initialized from something else than return/argument type
-     checks .  */
-  callee = gimple_call_fndecl (stmt);
-  if (callee
-      && !gimple_check_call_matching_types (stmt, callee))
-    gimple_call_set_cannot_inline (stmt, true);
-
   if (inplace)
     return changed;
 
   /* Check for builtins that CCP can handle using information not
      available in the generic fold routines.  */
+  callee = gimple_call_fndecl (stmt);
   if (callee && DECL_BUILT_IN (callee))
     {
       tree result = gimple_fold_builtin (stmt);
index 1facb3296f6e74c1279cba2f42a56884f6ee45dd..dc112d0dd934777cbee7498d4ee358f5b0690917 100644 (file)
@@ -219,18 +219,11 @@ input_gimple_stmt (struct lto_input_block *ib, struct data_in *data_in,
        }
       if (is_gimple_call (stmt))
        {
-         tree fndecl;
          if (gimple_call_internal_p (stmt))
            gimple_call_set_internal_fn
              (stmt, streamer_read_enum (ib, internal_fn, IFN_LAST));
          else
            gimple_call_set_fntype (stmt, stream_read_tree (ib, data_in));
-         /* Update the non-inlinable flag conservatively.  */
-         fndecl = gimple_call_fndecl (stmt);
-         if (fndecl
-             && !gimple_call_cannot_inline_p (stmt)
-             && !gimple_check_call_matching_types (stmt, fndecl))
-           gimple_call_set_cannot_inline (stmt, true);
        }
       break;
 
index 071c6515b4cae3f008672714d3937b890968bf71..c72b39a0d64597c7c7909da91d639b645f4849d0 100644 (file)
@@ -370,7 +370,6 @@ gimple_build_call_from_tree (tree t)
   /* Carry all the CALL_EXPR flags to the new GIMPLE_CALL.  */
   gimple_call_set_chain (call, CALL_EXPR_STATIC_CHAIN (t));
   gimple_call_set_tail (call, CALL_EXPR_TAILCALL (t));
-  gimple_call_set_cannot_inline (call, CALL_CANNOT_INLINE_P (t));
   gimple_call_set_return_slot_opt (call, CALL_EXPR_RETURN_SLOT_OPT (t));
   if (fndecl
       && DECL_BUILT_IN_CLASS (fndecl) == BUILT_IN_NORMAL
index 8536c70e87e21bdc421cbf90026bb346cb701744..ec4e76e13ddf237e0df77e4f206cdc9b58845ebd 100644 (file)
@@ -97,14 +97,13 @@ enum gimple_rhs_class
 enum gf_mask {
     GF_ASM_INPUT               = 1 << 0,
     GF_ASM_VOLATILE            = 1 << 1,
-    GF_CALL_CANNOT_INLINE      = 1 << 0,
-    GF_CALL_FROM_THUNK         = 1 << 1,
-    GF_CALL_RETURN_SLOT_OPT    = 1 << 2,
-    GF_CALL_TAILCALL           = 1 << 3,
-    GF_CALL_VA_ARG_PACK                = 1 << 4,
-    GF_CALL_NOTHROW            = 1 << 5,
-    GF_CALL_ALLOCA_FOR_VAR     = 1 << 6,
-    GF_CALL_INTERNAL           = 1 << 7,
+    GF_CALL_FROM_THUNK         = 1 << 0,
+    GF_CALL_RETURN_SLOT_OPT    = 1 << 1,
+    GF_CALL_TAILCALL           = 1 << 2,
+    GF_CALL_VA_ARG_PACK                = 1 << 3,
+    GF_CALL_NOTHROW            = 1 << 4,
+    GF_CALL_ALLOCA_FOR_VAR     = 1 << 5,
+    GF_CALL_INTERNAL           = 1 << 6,
     GF_OMP_PARALLEL_COMBINED   = 1 << 0,
 
     /* True on an GIMPLE_OMP_RETURN statement if the return does not require
@@ -2343,29 +2342,6 @@ gimple_call_tail_p (gimple s)
 }
 
 
-/* Set the inlinable status of GIMPLE_CALL S to INLINABLE_P.  */
-
-static inline void
-gimple_call_set_cannot_inline (gimple s, bool inlinable_p)
-{
-  GIMPLE_CHECK (s, GIMPLE_CALL);
-  if (inlinable_p)
-    s->gsbase.subcode |= GF_CALL_CANNOT_INLINE;
-  else
-    s->gsbase.subcode &= ~GF_CALL_CANNOT_INLINE;
-}
-
-
-/* Return true if GIMPLE_CALL S cannot be inlined.  */
-
-static inline bool
-gimple_call_cannot_inline_p (gimple s)
-{
-  GIMPLE_CHECK (s, GIMPLE_CALL);
-  return (s->gsbase.subcode & GF_CALL_CANNOT_INLINE) != 0;
-}
-
-
 /* If RETURN_SLOT_OPT_P is true mark GIMPLE_CALL S as valid for return
    slot optimization.  This transformation uses the target of the call
    expansion as the return slot for calls that return in memory.  */
index cfe6696f59040ecf091ec8e387ff933bb2f5b8bc..f548e703390523fbcc23aeca031d1b9ed94e6951 100644 (file)
@@ -2449,7 +2449,6 @@ gimplify_call_expr (tree *expr_p, gimple_seq *pre_p, bool want_value)
          CALL_EXPR_RETURN_SLOT_OPT (*expr_p)
            = CALL_EXPR_RETURN_SLOT_OPT (call);
          CALL_FROM_THUNK_P (*expr_p) = CALL_FROM_THUNK_P (call);
-         CALL_CANNOT_INLINE_P (*expr_p) = CALL_CANNOT_INLINE_P (call);
          SET_EXPR_LOCATION (*expr_p, EXPR_LOCATION (call));
          TREE_BLOCK (*expr_p) = TREE_BLOCK (call);
 
index 9ff247b103a2a974efbe4bc94a91de537b44ad30..21cbecd5cb66d22d586c8db613d3ec2a206ab330 100644 (file)
@@ -1249,7 +1249,7 @@ initialize_inline_failed (struct cgraph_edge *e)
     e->inline_failed = CIF_BODY_NOT_AVAILABLE;
   else if (callee->local.redefined_extern_inline)
     e->inline_failed = CIF_REDEFINED_EXTERN_INLINE;
-  else if (e->call_stmt && gimple_call_cannot_inline_p (e->call_stmt))
+  else if (e->call_stmt_cannot_inline_p)
     e->inline_failed = CIF_MISMATCHED_ARGUMENTS;
   else
     e->inline_failed = CIF_FUNCTION_NOT_CONSIDERED;
index e3c6b3ce824fe43d08dc36725e9c2e3d9bccff93..59852b5d26b0a3a2680419cd3f161962a7a1cf1f 100644 (file)
@@ -246,14 +246,6 @@ can_inline_edge_p (struct cgraph_edge *e, bool report)
   struct function *caller_cfun = DECL_STRUCT_FUNCTION (e->caller->decl);
   struct function *callee_cfun
     = callee ? DECL_STRUCT_FUNCTION (callee->decl) : NULL;
-  bool call_stmt_cannot_inline_p;
-
-  /* If E has a call statement in it, use the inline attribute from
-     the statement, otherwise use the inline attribute in E.  Edges
-     will not have statements when working in WPA mode.  */
-  call_stmt_cannot_inline_p = (e->call_stmt)
-                             ? gimple_call_cannot_inline_p (e->call_stmt)
-                             : e->call_stmt_cannot_inline_p;
 
   if (!caller_cfun && e->caller->clone_of)
     caller_cfun = DECL_STRUCT_FUNCTION (e->caller->clone_of->decl);
@@ -278,7 +270,7 @@ can_inline_edge_p (struct cgraph_edge *e, bool report)
       e->inline_failed = CIF_OVERWRITABLE;
       return false;
     }
-  else if (call_stmt_cannot_inline_p)
+  else if (e->call_stmt_cannot_inline_p)
     {
       e->inline_failed = CIF_MISMATCHED_ARGUMENTS;
       inlinable = false;
@@ -1957,8 +1949,10 @@ early_inliner (void)
                = estimate_num_insns (edge->call_stmt, &eni_size_weights);
              es->call_stmt_time
                = estimate_num_insns (edge->call_stmt, &eni_time_weights);
-             edge->call_stmt_cannot_inline_p
-               = gimple_call_cannot_inline_p (edge->call_stmt);
+             if (edge->callee->decl
+                 && !gimple_check_call_matching_types (edge->call_stmt,
+                                                       edge->callee->decl))
+               edge->call_stmt_cannot_inline_p = true;
            }
          timevar_pop (TV_INTEGRATION);
          iterations++;
index 69f0453ee4f58d951cb80c699a71ab454c5cb9a7..25ae3868c314c8c1d023dfd062ee50bbf014f1ff 100644 (file)
@@ -1905,13 +1905,10 @@ update_indirect_edges_after_inlining (struct cgraph_edge *cs,
       if (new_direct_edge)
        {
          new_direct_edge->indirect_inlining_edge = 1;
-         if (new_direct_edge->call_stmt
-             && !gimple_check_call_matching_types (new_direct_edge->call_stmt,
-                                                   new_direct_edge->callee->decl))
-           {
-             gimple_call_set_cannot_inline (new_direct_edge->call_stmt, true);
-             new_direct_edge->call_stmt_cannot_inline_p = true;
-           }
+         if (new_direct_edge->call_stmt)
+           new_direct_edge->call_stmt_cannot_inline_p
+             = !gimple_check_call_matching_types (new_direct_edge->call_stmt,
+                                                  new_direct_edge->callee->decl);
          if (new_edges)
            {
              VEC_safe_push (cgraph_edge_p, heap, *new_edges,
@@ -2577,9 +2574,6 @@ ipa_modify_call_arguments (struct cgraph_edge *cs, gimple stmt,
     gimple_set_location (new_stmt, gimple_location (stmt));
   gimple_call_set_chain (new_stmt, gimple_call_chain (stmt));
   gimple_call_copy_flags (new_stmt, stmt);
-  if (gimple_call_cannot_inline_p (stmt))
-    gimple_call_set_cannot_inline
-      (new_stmt, !gimple_check_call_matching_types (new_stmt, callee_decl));
 
   if (dump_file && (dump_flags & TDF_DETAILS))
     {
index 9479aebfb342153eec60ec74f9d37b6d556331f9..67fea759459f40b9d542e444d3fa4d0d59aefd62 100644 (file)
@@ -929,7 +929,6 @@ mf_xform_derefs_1 (gimple_stmt_iterator *iter, tree *tp,
 }
 /* Transform
    1) Memory references.
-   2) BUILTIN_ALLOCA calls.
 */
 static void
 mf_xform_statements (void)
@@ -970,16 +969,6 @@ mf_xform_statements (void)
                 }
               break;
 
-            case GIMPLE_CALL:
-              {
-                tree fndecl = gimple_call_fndecl (s);
-                if (fndecl && (DECL_FUNCTION_CODE (fndecl) == BUILT_IN_ALLOCA
-                              || (DECL_FUNCTION_CODE (fndecl)
-                                  == BUILT_IN_ALLOCA_WITH_ALIGN)))
-                  gimple_call_set_cannot_inline (s, true);
-              }
-              break;
-
             default:
               ;
             }
index fa4adac5d80ceda6858a34b919221147fa790566..4544f1ad126bb695aac45ef5d6d4b22a04100135 100644 (file)
@@ -533,9 +533,6 @@ struct GTY(()) tree_common {
        CASE_HIGH_SEEN in
            CASE_LABEL_EXPR
 
-       CALL_CANNOT_INLINE_P in
-           CALL_EXPR
        ENUM_IS_SCOPED in
           ENUMERAL_TYPE
 
@@ -1245,9 +1242,6 @@ extern void omp_clause_range_check_failed (const_tree, const char *, int,
 #define CASE_HIGH_SEEN(NODE) \
   (CASE_LABEL_EXPR_CHECK (NODE)->base.static_flag)
 
-/* Used to mark a CALL_EXPR as not suitable for inlining.  */
-#define CALL_CANNOT_INLINE_P(NODE) (CALL_EXPR_CHECK (NODE)->base.static_flag)
-
 /* Used to mark scoped enums.  */
 #define ENUM_IS_SCOPED(NODE) (ENUMERAL_TYPE_CHECK (NODE)->base.static_flag)
 
This page took 0.132593 seconds and 5 git commands to generate.