lto-cgraph.c (lto_output_varpool_node): Always output constant pool references.

* lto-cgraph.c (lto_output_varpool_node): Always output constant pool
	references.

	* lto.c: Do not attempt to make constant pool references global.

From-SVN: r158938
This commit is contained in:
Jan Hubicka 2010-04-30 16:26:51 +02:00 committed by Jan Hubicka
parent 9c6e079827
commit 05575e0780
4 changed files with 30 additions and 6 deletions

View File

@ -1,3 +1,8 @@
2010-04-30 Jan Hubicka <jh@suse.cz>
* lto-cgraph.c (lto_output_varpool_node): Always output constant pool
references.
2010-04-30 Jan Hubicka <jh@suse.cz>
* tree-profile.c (tree_init_ic_make_global_vars): Mark new decls as

View File

@ -374,10 +374,21 @@ lto_output_varpool_node (struct lto_simple_output_block *ob, struct varpool_node
gcc_assert (node->finalized || !node->analyzed);
gcc_assert (node->needed);
gcc_assert (!node->alias);
/* FIXME: We have no idea how we move references around. For moment assume that
everything is used externally. */
bp_pack_value (bp, flag_wpa, 1); /* used_from_other_parition. */
bp_pack_value (bp, boundary_p, 1); /* in_other_partition. */
/* Constant pool initializers can be de-unified into individual ltrans units.
FIXME: Alternatively at -Os we may want to avoid generating for them the local
labels and share them across LTRANS partitions. */
if (DECL_IN_CONSTANT_POOL (node->decl))
{
bp_pack_value (bp, 0, 1); /* used_from_other_parition. */
bp_pack_value (bp, 0, 1); /* in_other_partition. */
}
else
{
/* FIXME: We have no idea how we move references around. For moment assume that
everything is used externally. */
bp_pack_value (bp, flag_wpa, 1); /* used_from_other_parition. */
bp_pack_value (bp, boundary_p, 1); /* in_other_partition. */
}
/* Also emit any extra name aliases. */
for (alias = node->extra_name; alias; alias = alias->next)
count++;

View File

@ -1,3 +1,7 @@
2010-04-30 Jan Hubicka <jh@suse.cz>
* lto.c: Do not attempt to make constant pool references global.
2010-04-28 Jan Hubicka <jh@suse.cz>
* lto/lto.c (lto_read_in_decl_state): Use GGC.

View File

@ -723,9 +723,13 @@ lto_promote_cross_file_statics (void)
gcc_assert (flag_wpa);
/* At moment we make no attempt to figure out who is refering the variables,
so all must become global. */
so all must become global.
Constant pool references use internal labels and thus can not be made global.
It is sensible to keep those ltrans local to allow better optimization. */
for (vnode = varpool_nodes; vnode; vnode = vnode->next)
if (!vnode->externally_visible && vnode->analyzed)
if (!vnode->externally_visible && vnode->analyzed
&& !DECL_IN_CONSTANT_POOL (vnode->decl))
{
TREE_PUBLIC (vnode->decl) = 1;
DECL_VISIBILITY (vnode->decl) = VISIBILITY_HIDDEN;