From: hubicka <hubicka@138bc75d-0d04-0410-961f-82ee72b054a4>
Date: Wed, 21 Apr 2010 14:41:03 +0000 (+0000)
Subject: 	* lto.c (globalize_cross_file_statics): When function has address taken,
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=e288c7a9d241b2ec4f17a2a9b3a058fc30c86725;p=thirdparty%2Fgcc.git

	* lto.c (globalize_cross_file_statics): When function has address taken,
	it needs to be public.
	* varpool.c (decide_is_variable_needed): Variable is always needed
	during ltrans.


git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@158609 138bc75d-0d04-0410-961f-82ee72b054a4
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 1fe4172587a6..13ac23956aaa 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2010-04-21  Jan Hubicka  <jh@suse.cz>
+
+	* varpool.c (decide_is_variable_needed): Variable is always needed
+	during ltrans.
+
 2010-04-21  Jan Hubicka  <jh@suse.cz>
 
 	* opts.c (decode_options): Enable pure-const pass for whopr.
diff --git a/gcc/lto/ChangeLog b/gcc/lto/ChangeLog
index 9118e0045ea7..a97314c928be 100644
--- a/gcc/lto/ChangeLog
+++ b/gcc/lto/ChangeLog
@@ -1,3 +1,8 @@
+2010-04-21  Jan Hubicka  <jh@suse.cz>
+
+	* lto.c (globalize_cross_file_statics): When function has address taken,
+	it needs to be public.
+
 2010-04-20  Jan Hubicka  <jh@suse.cz>
 
 	* lto.c (lto_add_inline_clones): Do not track inlined_decls.
diff --git a/gcc/lto/lto.c b/gcc/lto/lto.c
index 1544f05ab4fc..ea8f03a131db 100644
--- a/gcc/lto/lto.c
+++ b/gcc/lto/lto.c
@@ -760,7 +760,8 @@ globalize_cross_file_statics (tree *tp, int *walk_subtrees ATTRIBUTE_UNUSED,
     }
   else if (TREE_CODE (t) == FUNCTION_DECL && !TREE_PUBLIC (t))
     {
-      if (!cgraph_node_in_set_p (cgraph_node (t), context->set))
+      if (!cgraph_node_in_set_p (cgraph_node (t), context->set)
+	  || cgraph_node (t)->address_taken)
 	{
 	  /* This file-scope static function is reachable from a set
 	     which does not contain the function DECL.  Make it global
diff --git a/gcc/varpool.c b/gcc/varpool.c
index a13742d4957a..40decfc867b5 100644
--- a/gcc/varpool.c
+++ b/gcc/varpool.c
@@ -230,6 +230,12 @@ varpool_reset_queue (void)
 bool
 decide_is_variable_needed (struct varpool_node *node, tree decl)
 {
+  /* We do not track variable references at all and thus have no idea if the
+     variable was referenced in some other partition or not.  
+     FIXME: We really need address taken edges in callgraph and varpool to
+     drive WPA and decide whether other partition might reference it or not.  */
+  if (flag_ltrans)
+    return true;
   /* If the user told us it is used, then it must be so.  */
   if ((node->externally_visible && !DECL_COMDAT (decl))
       || node->force_output)