Commit dfa15de2 by Sterling Augustine Committed by Sterling Augustine

dwarf2out.c (add_pubname): Add comment.

2012-06-29  Sterling Augustine  <saugustine@google.com>

	* dwarf2out.c (add_pubname): Add comment.
	(add_pubtype): Fix indentation.
	(gen_enumeration_type_die): Likewise.

From-SVN: r189084
parent 271be516
2012-06-29 Sterling Augustine <saugustine@google.com>
* dwarf2out.c (add_pubname): Add comment.
(add_pubtype): Fix indentation.
(gen_enumeration_type_die): Likewise.
2012-06-29 Jakub Jelinek <jakub@redhat.com> 2012-06-29 Jakub Jelinek <jakub@redhat.com>
* tree-vect-generic.c (expand_vector_divmod): For even/odd * tree-vect-generic.c (expand_vector_divmod): For even/odd
......
...@@ -8168,6 +8168,11 @@ add_pubname (tree decl, dw_die_ref die) ...@@ -8168,6 +8168,11 @@ add_pubname (tree decl, dw_die_ref die)
if (!want_pubnames ()) if (!want_pubnames ())
return; return;
/* Don't add items to the table when we expect that the consumer will have
just read the enclosing die. For example, if the consumer is looking at a
class_member, it will either be inside the class already, or will have just
looked up the class to find the member. Either way, searching the class is
faster than searching the index. */
if ((TREE_PUBLIC (decl) && !is_class_die (die->die_parent)) if ((TREE_PUBLIC (decl) && !is_class_die (die->die_parent))
|| is_cu_die (die->die_parent) || is_namespace_die (die->die_parent)) || is_cu_die (die->die_parent) || is_namespace_die (die->die_parent))
{ {
...@@ -8212,11 +8217,11 @@ add_pubtype (tree decl, dw_die_ref die) ...@@ -8212,11 +8217,11 @@ add_pubtype (tree decl, dw_die_ref die)
scope = TYPE_P (decl) ? TYPE_CONTEXT (decl) : NULL; scope = TYPE_P (decl) ? TYPE_CONTEXT (decl) : NULL;
if (scope && TREE_CODE (scope) == NAMESPACE_DECL) if (scope && TREE_CODE (scope) == NAMESPACE_DECL)
{ {
scope_name = lang_hooks.dwarf_name (scope, 1); scope_name = lang_hooks.dwarf_name (scope, 1);
if (scope_name != NULL && scope_name[0] != '\0') if (scope_name != NULL && scope_name[0] != '\0')
scope_name = concat (scope_name, sep, NULL); scope_name = concat (scope_name, sep, NULL);
else else
scope_name = ""; scope_name = "";
} }
...@@ -8231,8 +8236,8 @@ add_pubtype (tree decl, dw_die_ref die) ...@@ -8231,8 +8236,8 @@ add_pubtype (tree decl, dw_die_ref die)
{ {
e.die = die; e.die = die;
e.name = concat (scope_name, name, NULL); e.name = concat (scope_name, name, NULL);
VEC_safe_push (pubname_entry, gc, pubtype_table, &e); VEC_safe_push (pubname_entry, gc, pubtype_table, &e);
} }
/* Although it might be more consistent to add the pubinfo for the /* Although it might be more consistent to add the pubinfo for the
enumerators as their dies are created, they should only be added if the enumerators as their dies are created, they should only be added if the
...@@ -16282,7 +16287,7 @@ gen_enumeration_type_die (tree type, dw_die_ref context_die) ...@@ -16282,7 +16287,7 @@ gen_enumeration_type_die (tree type, dw_die_ref context_die)
else else
add_AT_flag (type_die, DW_AT_declaration, 1); add_AT_flag (type_die, DW_AT_declaration, 1);
add_pubtype (type, type_die); add_pubtype (type, type_die);
return type_die; return type_die;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment