Now with the patch attached.

On Thu, Nov 14, 2013 at 9:49 PM, Uros Bizjak <ubiz...@gmail.com> wrote:
> On Thu, Nov 14, 2013 at 7:16 PM, Uros Bizjak <ubiz...@gmail.com> wrote:
>
> Attached patch fixes lto-profiledbootstrap, introduced by:
>
>>>>> 2013-11-12  Martin Jambor  <mjam...@suse.cz>
>>>>>
>>>>> * cgraph.c (cgraph_get_create_node): Do what
>>>>> cgraph_get_create_real_symbol_node used to do.
>>>>> (cgraph_get_create_real_symbol_node): Removed.  Changed all users to
>>>>> call cgraph_get_create_node.
>>>>> * cgraph.h (cgraph_get_create_real_symbol_node): Removed.
>>>>> * lto-streamer-in.c (input_function): Call cgraph_get_node instead of
>>>>> cgraph_get_create_node.  Assert we get a node.
>
> The patch reverts lto-streamer-in.c functionality back to what was
> doing before the above patch.
>
> 2013-11-14  Uros Bizjak  <ubiz...@gmail.com>
>
>     * lto-streamer-in.c (input function): Call cgraph_create_node if
>     cgraph_get_node failed.
>
> Tested with lto-profiledbootstrap on x86_64-pc-linux-gnu, regression
> tested also with -m32 [1].
>
> OK for mainline?
>
> [1] http://gcc.gnu.org/ml/gcc-testresults/2013-11/msg00989.html
>
> Uros.
Index: lto-streamer-in.c
===================================================================
--- lto-streamer-in.c   (revision 204807)
+++ lto-streamer-in.c   (working copy)
@@ -917,7 +917,8 @@ input_function (tree fn_decl, struct data_in *data
   gimple_register_cfg_hooks ();
 
   node = cgraph_get_node (fn_decl);
-  gcc_checking_assert (node);
+  if (!node)
+    node = cgraph_create_node (fn_decl);
   input_struct_function_base (fn, data_in, ib);
   input_cfg (ib_cfg, fn, node->count_materialization_scale);
 

Reply via email to