On 08/01/14 05:35, Martin Liška wrote:
2014-07-31 Martin Liska<mli...@suse.cz>
* cgraph.h (cgraph_node_set, varpool_node_set): Removed structs and all
related functions that manipulated these data structures.
(cgraph_new_nodes): vec replaces cgraph_node_set.
* cgraphunit.c (cgraph_new_nodes): Likewise.
(cgraph_process_new_functions): Likewise.
(cgraph_node::add_new_function): Likewise.
* ipa-inline-transform.c (can_remove_node_now_p_1): Likewise.
* ipa-utils.c: Implementations of legacy data structures removed,
data structure was replaced by a hash_map.
* tree-emutls.c (tls_vars): hash_map replaces varpool_node_set.
(emutls_index): New functions are used.
(ipa_lower_emutls): Likewise.
Looks like a good cleanup. Just aone nit left with the most recent
version (the one where you fixed the indention issues pointed out by Jakub):
symtab-node-removal.patch
diff --git a/gcc/tree-emutls.c b/gcc/tree-emutls.c
index 89197c7..2c9cfe4 100644
--- a/gcc/tree-emutls.c
+++ b/gcc/tree-emutls.c
@@ -608,13 +581,20 @@ lower_emutls_phi_arg (gimple phi, unsigned int i, struct
lower_emutls_data *d)
}
}
+bool
+reset_access (varpool_node * const &, tls_var_data *data, void *)
+{
+ data->access = NULL;
+
+ return true;
+}
Need a comment for this function.
Approved once you add that comment. Please post the final patch for
archival purposes.
Thanks,
Jeff
