]> git.ipfire.org Git - thirdparty/binutils-gdb.git/blobdiff - gdb/cp-namespace.c
Update copyright year range in all GDB files
[thirdparty/binutils-gdb.git] / gdb / cp-namespace.c
index 030cfb9a4f0ff62e654df0a7ea1eaeb7cabac26d..0751fda7e6360a1712ab24c5905625a161903d66 100644 (file)
@@ -1,6 +1,5 @@
 /* Helper routines for C++ support in GDB.
-   Copyright (C) 2003, 2004, 2007, 2008, 2009, 2010, 2011
-   Free Software Foundation, Inc.
+   Copyright (C) 2003-2018 Free Software Foundation, Inc.
 
    Contributed by David Carlton and by Kealia, Inc.
 
@@ -24,7 +23,6 @@
 #include "gdb_obstack.h"
 #include "symtab.h"
 #include "symfile.h"
-#include "gdb_assert.h"
 #include "block.h"
 #include "objfiles.h"
 #include "gdbtypes.h"
 #include "frame.h"
 #include "buildsym.h"
 #include "language.h"
+#include "namespace.h"
+#include <string>
 
-static struct symbol *lookup_namespace_scope (const char *name,
-                                             const struct block *block,
-                                             const domain_enum domain,
-                                             const char *scope,
-                                             int scope_len);
-
-static struct symbol *lookup_symbol_file (const char *name,
-                                         const struct block *block,
-                                         const domain_enum domain,
-                                         int anonymous_namespace);
+static struct block_symbol
+  cp_lookup_nested_symbol_1 (struct type *container_type,
+                            const char *nested_name,
+                            const char *concatenated_name,
+                            const struct block *block,
+                            const domain_enum domain,
+                            int basic_lookup, int is_in_anonymous);
 
 static struct type *cp_lookup_transparent_type_loop (const char *name,
                                                     const char *scope,
@@ -52,12 +49,9 @@ static struct type *cp_lookup_transparent_type_loop (const char *name,
 /* Check to see if SYMBOL refers to an object contained within an
    anonymous namespace; if so, add an appropriate using directive.  */
 
-/* Optimize away strlen ("(anonymous namespace)").  */
-
-#define ANONYMOUS_NAMESPACE_LEN 21
-
 void
-cp_scan_for_anonymous_namespaces (const struct symbol *symbol)
+cp_scan_for_anonymous_namespaces (const struct symbol *const symbol,
+                                 struct objfile *const objfile)
 {
   if (SYMBOL_DEMANGLED_NAME (symbol) != NULL)
     {
@@ -68,7 +62,7 @@ cp_scan_for_anonymous_namespaces (const struct symbol *symbol)
       /* Start with a quick-and-dirty check for mention of "(anonymous
         namespace)".  */
 
-      if (!cp_is_anonymous (name))
+      if (!cp_is_in_anonymous (name))
        return;
 
       previous_component = 0;
@@ -76,17 +70,18 @@ cp_scan_for_anonymous_namespaces (const struct symbol *symbol)
 
       while (name[next_component] == ':')
        {
-         if ((next_component - previous_component) == ANONYMOUS_NAMESPACE_LEN
+         if (((next_component - previous_component)
+              == CP_ANONYMOUS_NAMESPACE_LEN)
              && strncmp (name + previous_component,
-                         "(anonymous namespace)",
-                         ANONYMOUS_NAMESPACE_LEN) == 0)
+                         CP_ANONYMOUS_NAMESPACE_STR,
+                         CP_ANONYMOUS_NAMESPACE_LEN) == 0)
            {
              int dest_len = (previous_component == 0
                              ? 0 : previous_component - 2);
              int src_len = next_component;
 
-             char *dest = alloca (dest_len + 1);
-             char *src = alloca (src_len + 1);
+             char *dest = (char *) alloca (dest_len + 1);
+             char *src = (char *) alloca (src_len + 1);
 
              memcpy (dest, name, dest_len);
              memcpy (src, name, src_len);
@@ -98,8 +93,10 @@ cp_scan_for_anonymous_namespaces (const struct symbol *symbol)
                 anonymous namespace.  So add symbols in it to the
                 namespace given by the previous component if there is
                 one, or to the global namespace if there isn't.  */
-             cp_add_using_directive (dest, src, NULL, NULL,
-                                     &SYMBOL_SYMTAB (symbol)->objfile->objfile_obstack);
+             std::vector<const char *> excludes;
+             add_using_directive (&local_using_directives,
+                                  dest, src, NULL, NULL, excludes, 1,
+                                  &objfile->objfile_obstack);
            }
          /* The "+ 2" is for the "::".  */
          previous_component = next_component + 2;
@@ -110,166 +107,235 @@ cp_scan_for_anonymous_namespaces (const struct symbol *symbol)
     }
 }
 
+/* Test whether or not NAMESPACE looks like it mentions an anonymous
+   namespace; return nonzero if so.  */
 
-/* Add a using directive to using_directives.  If the using directive
-   in question has already been added, don't add it twice.
+int
+cp_is_in_anonymous (const char *symbol_name)
+{
+  return (strstr (symbol_name, CP_ANONYMOUS_NAMESPACE_STR)
+         != NULL);
+}
 
-   Create a new struct using_direct which imports the namespace SRC
-   into the scope DEST.  ALIAS is the name of the imported namespace
-   in the current scope.  If ALIAS is NULL then the namespace is known
-   by its original name.  DECLARATION is the name if the imported
-   varable if this is a declaration import (Eg. using A::x), otherwise
-   it is NULL.  The arguments are copied into newly allocated memory
-   so they can be temporaries.  */
+/* Look up NAME in DOMAIN in BLOCK's static block and in global blocks.
+   If IS_IN_ANONYMOUS is nonzero, the symbol in question is located
+   within an anonymous namespace.  */
 
-void
-cp_add_using_directive (const char *dest,
-                       const char *src,
-                       const char *alias,
-                       const char *declaration,
-                        struct obstack *obstack)
+static struct block_symbol
+cp_basic_lookup_symbol (const char *name, const struct block *block,
+                       const domain_enum domain, int is_in_anonymous)
 {
-  struct using_direct *current;
-  struct using_direct *new;
-  
-  /* Has it already been added?  */
+  struct block_symbol sym;
 
-  for (current = using_directives; current != NULL; current = current->next)
+  sym = lookup_symbol_in_static_block (name, block, domain);
+  if (sym.symbol != NULL)
+    return sym;
+
+  if (is_in_anonymous)
     {
-      if (strcmp (current->import_src, src) == 0
-          && strcmp (current->import_dest, dest) == 0
-          && ((alias == NULL && current->alias == NULL)
-              || (alias != NULL && current->alias != NULL
-                 && strcmp (alias, current->alias) == 0))
-         && ((declaration == NULL && current->declaration == NULL)
-             || (declaration != NULL && current->declaration != NULL
-                 && strcmp (declaration, current->declaration) == 0)))
-       return;
-    }
+      /* Symbols defined in anonymous namespaces have external linkage
+        but should be treated as local to a single file nonetheless.
+        So we only search the current file's global block.  */
 
-  new = OBSTACK_ZALLOC (obstack, struct using_direct);
+      const struct block *global_block = block_global_block (block);
 
-  new->import_src = obsavestring (src, strlen (src), obstack);
-  new->import_dest = obsavestring (dest, strlen (dest), obstack);
+      if (global_block != NULL)
+       {
+         sym.symbol = lookup_symbol_in_block (name, global_block, domain);
+         sym.block = global_block;
+       }
+    }
+  else
+    sym = lookup_global_symbol (name, block, domain);
 
-  if (alias != NULL)
-    new->alias = obsavestring (alias, strlen (alias), obstack);
+  return sym;
+}
 
-  if (declaration != NULL)
-    new->declaration = obsavestring (declaration, strlen (declaration),
-                                     obstack);
+/* Search bare symbol NAME in DOMAIN in BLOCK.
+   NAME is guaranteed to not have any scope (no "::") in its name, though
+   if for example NAME is a template spec then "::" may appear in the
+   argument list.
+   If LANGDEF is non-NULL then try to lookup NAME as a primitive type in
+   that language.  Normally we wouldn't need LANGDEF but fortran also uses
+   this code.
+   If SEARCH is non-zero then see if we can determine "this" from BLOCK, and
+   if so then also search for NAME in that class.  */
+
+static struct block_symbol
+cp_lookup_bare_symbol (const struct language_defn *langdef,
+                      const char *name, const struct block *block,
+                      const domain_enum domain, int search)
+{
+  struct block_symbol sym;
 
-  new->next = using_directives;
-  using_directives = new;
-}
+  /* Note: We can't do a simple assert for ':' not being in NAME because
+     ':' may be in the args of a template spec.  This isn't intended to be
+     a complete test, just cheap and documentary.  */
+  if (strchr (name, '<') == NULL && strchr (name, '(') == NULL)
+    gdb_assert (strstr (name, "::") == NULL);
 
-/* Record the namespace that the function defined by SYMBOL was
-   defined in, if necessary.  BLOCK is the associated block; use
-   OBSTACK for allocation.  */
+  sym = lookup_symbol_in_static_block (name, block, domain);
+  if (sym.symbol != NULL)
+    return sym;
 
-void
-cp_set_block_scope (const struct symbol *symbol,
-                   struct block *block,
-                   struct obstack *obstack,
-                   const char *processing_current_prefix,
-                   int processing_has_namespace_info)
-{
-  if (processing_has_namespace_info)
+  /* If we didn't find a definition for a builtin type in the static block,
+     search for it now.  This is actually the right thing to do and can be
+     a massive performance win.  E.g., when debugging a program with lots of
+     shared libraries we could search all of them only to find out the
+     builtin type isn't defined in any of them.  This is common for types
+     like "void".  */
+  if (langdef != NULL && domain == VAR_DOMAIN)
     {
-      block_set_scope
-       (block, obsavestring (processing_current_prefix,
-                             strlen (processing_current_prefix),
-                             obstack),
-        obstack);
+      struct gdbarch *gdbarch;
+
+      if (block == NULL)
+       gdbarch = target_gdbarch ();
+      else
+       gdbarch = block_gdbarch (block);
+      sym.symbol
+       = language_lookup_primitive_type_as_symbol (langdef, gdbarch, name);
+      sym.block = NULL;
+      if (sym.symbol != NULL)
+       return sym;
     }
-  else if (SYMBOL_DEMANGLED_NAME (symbol) != NULL)
-    {
-      /* Try to figure out the appropriate namespace from the
-        demangled name.  */
 
-      /* FIXME: carlton/2003-04-15: If the function in question is
-        a method of a class, the name will actually include the
-        name of the class as well.  This should be harmless, but
-        is a little unfortunate.  */
+  sym = lookup_global_symbol (name, block, domain);
+  if (sym.symbol != NULL)
+    return sym;
 
-      const char *name = SYMBOL_DEMANGLED_NAME (symbol);
-      unsigned int prefix_len = cp_entire_prefix_len (name);
+  if (search)
+    {
+      struct block_symbol lang_this;
+      struct type *type;
 
-      block_set_scope (block,
-                      obsavestring (name, prefix_len, obstack),
-                      obstack);
-    }
-}
+      lang_this.symbol = NULL;
 
-/* Test whether or not NAMESPACE looks like it mentions an anonymous
-   namespace; return nonzero if so.  */
+      if (langdef != NULL)
+       lang_this = lookup_language_this (langdef, block);
 
-int
-cp_is_anonymous (const char *namespace)
-{
-  return (strstr (namespace, "(anonymous namespace)")
-         != NULL);
-}
+      if (lang_this.symbol == NULL)
+       return null_block_symbol;
 
-/* The C++-specific version of name lookup for static and global
-   names.  This makes sure that names get looked for in all namespaces
-   that are in scope.  NAME is the natural name of the symbol that
-   we're looking for, BLOCK is the block that we're searching within,
-   DOMAIN says what kind of symbols we're looking for, and if SYMTAB
-   is non-NULL, we should store the symtab where we found the symbol
-   in it.  */
 
-struct symbol *
-cp_lookup_symbol_nonlocal (const char *name,
-                          const struct block *block,
-                          const domain_enum domain)
-{
-  struct symbol *sym;
-  const char *scope = block_scope (block);
+      type = check_typedef (TYPE_TARGET_TYPE (SYMBOL_TYPE (lang_this.symbol)));
+      /* If TYPE_NAME is NULL, abandon trying to find this symbol.
+        This can happen for lambda functions compiled with clang++,
+        which outputs no name for the container class.  */
+      if (TYPE_NAME (type) == NULL)
+       return null_block_symbol;
 
-  sym = lookup_namespace_scope (name, block,
-                               domain, scope, 0);
-  if (sym != NULL)
-    return sym;
+      /* Look for symbol NAME in this class.  */
+      sym = cp_lookup_nested_symbol (type, name, block, domain);
+    }
 
-  return cp_lookup_symbol_namespace (scope, name,
-                                    block, domain);
+  return sym;
 }
 
-/* Look up NAME in the C++ namespace NAMESPACE.  Other arguments are
-   as in cp_lookup_symbol_nonlocal.  */
+/* Search NAME in DOMAIN in all static blocks, and then in all baseclasses.
+   BLOCK specifies the context in which to perform the search.
+   NAME is guaranteed to have scope (contain "::") and PREFIX_LEN specifies
+   the length of the entire scope of NAME (up to, but not including, the last
+   "::".
+
+   Note: At least in the case of Fortran, which also uses this code, there
+   may be no text after the last "::".  */
+
+static struct block_symbol
+cp_search_static_and_baseclasses (const char *name,
+                                 const struct block *block,
+                                 const domain_enum domain,
+                                 unsigned int prefix_len,
+                                 int is_in_anonymous)
+{
+  /* Check for malformed input.  */
+  if (prefix_len + 2 > strlen (name) || name[prefix_len + 1] != ':')
+    return null_block_symbol;
+
+  /* The class, namespace or function name is everything up to and
+     including PREFIX_LEN.  */
+  std::string scope (name, prefix_len);
+
+  /* The rest of the name is everything else past the initial scope
+     operator.  */
+  const char *nested = name + prefix_len + 2;
+
+  /* Lookup the scope symbol.  If none is found, there is nothing more
+     that can be done.  SCOPE could be a namespace, so always look in
+     VAR_DOMAIN.  This works for classes too because of
+     symbol_matches_domain (which should be replaced with something
+     else, but it's what we have today).  */
+  block_symbol scope_sym = lookup_symbol_in_static_block (scope.c_str (),
+                                                         block, VAR_DOMAIN);
+  if (scope_sym.symbol == NULL)
+    scope_sym = lookup_global_symbol (scope.c_str (), block, VAR_DOMAIN);
+  if (scope_sym.symbol == NULL)
+    return null_block_symbol;
+
+  struct type *scope_type = SYMBOL_TYPE (scope_sym.symbol);
+
+  /* If the scope is a function/method, then look up NESTED as a local
+     static variable.  E.g., "print 'function()::static_var'".  */
+  if (TYPE_CODE (scope_type) == TYPE_CODE_FUNC
+      || TYPE_CODE (scope_type) == TYPE_CODE_METHOD)
+    return lookup_symbol (nested, SYMBOL_BLOCK_VALUE (scope_sym.symbol),
+                         VAR_DOMAIN, NULL);
+
+  /* Look for a symbol named NESTED in this class/namespace.
+     The caller is assumed to have already have done a basic lookup of NAME.
+     So we pass zero for BASIC_LOOKUP to cp_lookup_nested_symbol_1 here.  */
+  return cp_lookup_nested_symbol_1 (scope_type, nested, name,
+                                   block, domain, 0, is_in_anonymous);
+}
 
-static struct symbol *
-cp_lookup_symbol_in_namespace (const char *namespace,
-                               const char *name,
-                               const struct block *block,
-                               const domain_enum domain)
+/* Look up NAME in the C++ namespace NAMESPACE.  Other arguments are
+   as in cp_lookup_symbol_nonlocal.  If SEARCH is non-zero, search
+   through base classes for a matching symbol.
+
+   Note: Part of the complexity is because NAME may itself specify scope.
+   Part of the complexity is also because this handles the case where
+   there is no scoping in which case we also try looking in the class of
+   "this" if we can compute it.  */
+
+static struct block_symbol
+cp_lookup_symbol_in_namespace (const char *the_namespace, const char *name,
+                              const struct block *block,
+                              const domain_enum domain, int search)
 {
-  if (namespace[0] == '\0')
-    {
-      return lookup_symbol_file (name, block, domain, 0);
-    }
-  else
-    {
-      char *concatenated_name = alloca (strlen (namespace) + 2
-                                       + strlen (name) + 1);
+  char *concatenated_name = NULL;
+  int is_in_anonymous;
+  unsigned int prefix_len;
+  struct block_symbol sym;
 
-      strcpy (concatenated_name, namespace);
+  if (the_namespace[0] != '\0')
+    {
+      concatenated_name
+       = (char *) alloca (strlen (the_namespace) + 2 + strlen (name) + 1);
+      strcpy (concatenated_name, the_namespace);
       strcat (concatenated_name, "::");
       strcat (concatenated_name, name);
-      return lookup_symbol_file (concatenated_name, block, domain,
-                                cp_is_anonymous (namespace));
+      name = concatenated_name;
     }
-}
 
-/* Used for cleanups to reset the "searched" flag incase
-   of an error.  */
+  prefix_len = cp_entire_prefix_len (name);
+  if (prefix_len == 0)
+    return cp_lookup_bare_symbol (NULL, name, block, domain, search);
 
-static void
-reset_directive_searched (void *data)
-{
-  struct using_direct *direct = data;
-  direct->searched = 0;
+  /* This would be simpler if we just called cp_lookup_nested_symbol
+     at this point.  But that would require first looking up the containing
+     class/namespace.  Since we're only searching static and global blocks
+     there's often no need to first do that lookup.  */
+
+  is_in_anonymous
+    = the_namespace[0] != '\0' && cp_is_in_anonymous (the_namespace);
+  sym = cp_basic_lookup_symbol (name, block, domain, is_in_anonymous);
+  if (sym.symbol != NULL)
+    return sym;
+
+  if (search)
+    sym = cp_search_static_and_baseclasses (name, block, domain, prefix_len,
+                                           is_in_anonymous);
+
+  return sym;
 }
 
 /* Search for NAME by applying all import statements belonging to
@@ -295,28 +361,34 @@ reset_directive_searched (void *data)
 
    If SCOPE is "A::B" and SEARCH_PARENTS is true the imports of
    namespaces X and Y will be considered.  If SEARCH_PARENTS is false
-   only the import of Y is considered.  */
-
-struct symbol *
-cp_lookup_symbol_imports (const char *scope,
-                          const char *name,
-                          const struct block *block,
-                          const domain_enum domain,
-                          const int declaration_only,
-                          const int search_parents)
+   only the import of Y is considered.
+
+   SEARCH_SCOPE_FIRST is an internal implementation detail: Callers must
+   pass 0 for it.  Internally we pass 1 when recursing.  */
+
+static struct block_symbol
+cp_lookup_symbol_via_imports (const char *scope,
+                             const char *name,
+                             const struct block *block,
+                             const domain_enum domain,
+                             const int search_scope_first,
+                             const int declaration_only,
+                             const int search_parents)
 {
   struct using_direct *current;
-  struct symbol *sym = NULL;
+  struct block_symbol sym;
   int len;
   int directive_match;
-  struct cleanup *searched_cleanup;
 
-  /* First, try to find the symbol in the given namespace.  */
-  if (!declaration_only)
+  sym.symbol = NULL;
+  sym.block = NULL;
+
+  /* First, try to find the symbol in the given namespace if requested.  */
+  if (search_scope_first)
     sym = cp_lookup_symbol_in_namespace (scope, name,
-                                        block, domain);
-  
-  if (sym != NULL)
+                                        block, domain, 1);
+
+  if (sym.symbol != NULL)
     return sym;
 
   /* Go through the using directives.  If any of them add new names to
@@ -327,10 +399,11 @@ cp_lookup_symbol_imports (const char *scope,
        current != NULL;
        current = current->next)
     {
+      const char **excludep;
+
       len = strlen (current->import_dest);
       directive_match = (search_parents
-                         ? (strncmp (scope, current->import_dest,
-                                     strlen (current->import_dest)) == 0
+                         ? (startswith (scope, current->import_dest)
                             && (len == 0
                                 || scope[len] == ':'
                                || scope[len] == '\0'))
@@ -340,70 +413,70 @@ cp_lookup_symbol_imports (const char *scope,
          ancestors then it is applicable.  */
       if (directive_match && !current->searched)
        {
-       /* Mark this import as searched so that the recursive call
-           does not search it again.  */
-       current->searched = 1;
-       searched_cleanup = make_cleanup (reset_directive_searched,
-                                        current);
-
-       /* If there is an import of a single declaration, compare the
-          imported declaration (after optional renaming by its alias)
-          with the sought out name.  If there is a match pass
-          current->import_src as NAMESPACE to direct the search
-          towards the imported namespace.  */
-       if (current->declaration
-           && strcmp (name, current->alias
-                      ? current->alias : current->declaration) == 0)
-         sym = cp_lookup_symbol_in_namespace (current->import_src,
-                                              current->declaration,
-                                              block, domain);
-
-       /* If this is a DECLARATION_ONLY search or a symbol was found
-          or this import statement was an import declaration, the
-          search of this import is complete.  */
-        if (declaration_only || sym != NULL || current->declaration)
-          {
-            current->searched = 0;
-            discard_cleanups (searched_cleanup);
-
-            if (sym != NULL)
-              return sym;
-
-            continue;
-          }
-
-       if (current->alias != NULL
-           && strcmp (name, current->alias) == 0)
-         /* If the import is creating an alias and the alias matches
-            the sought name.  Pass current->import_src as the NAME to
-            direct the search towards the aliased namespace.  */
-         {
-           sym = cp_lookup_symbol_in_namespace (scope,
-                                                current->import_src,
-                                                block, domain);
-         }
-       else if (current->alias == NULL)
-         {
-           /* If this import statement creates no alias, pass
-               current->inner as NAMESPACE to direct the search
-               towards the imported namespace.  */
-           sym = cp_lookup_symbol_imports (current->import_src,
-                                           name, block,
-                                           domain, 0, 0);
-         }
-       current->searched = 0;
-       discard_cleanups (searched_cleanup);
+         /* Mark this import as searched so that the recursive call
+            does not search it again.  */
+         scoped_restore reset_directive_searched
+           = make_scoped_restore (&current->searched, 1);
+
+         /* If there is an import of a single declaration, compare the
+            imported declaration (after optional renaming by its alias)
+            with the sought out name.  If there is a match pass
+            current->import_src as NAMESPACE to direct the search
+            towards the imported namespace.  */
+         if (current->declaration
+             && strcmp (name, current->alias
+                        ? current->alias : current->declaration) == 0)
+           sym = cp_lookup_symbol_in_namespace (current->import_src,
+                                                current->declaration,
+                                                block, domain, 1);
+
+         /* If this is a DECLARATION_ONLY search or a symbol was found
+            or this import statement was an import declaration, the
+            search of this import is complete.  */
+         if (declaration_only || sym.symbol != NULL || current->declaration)
+           {
+             if (sym.symbol != NULL)
+               return sym;
+
+             continue;
+           }
 
-       if (sym != NULL)
-         return sym;
+         /* Do not follow CURRENT if NAME matches its EXCLUDES.  */
+         for (excludep = current->excludes; *excludep; excludep++)
+           if (strcmp (name, *excludep) == 0)
+             break;
+         if (*excludep)
+           continue;
+
+         if (current->alias != NULL
+             && strcmp (name, current->alias) == 0)
+           /* If the import is creating an alias and the alias matches
+              the sought name.  Pass current->import_src as the NAME to
+              direct the search towards the aliased namespace.  */
+           {
+             sym = cp_lookup_symbol_in_namespace (scope,
+                                                  current->import_src,
+                                                  block, domain, 1);
+           }
+         else if (current->alias == NULL)
+           {
+             /* If this import statement creates no alias, pass
+                current->inner as NAMESPACE to direct the search
+                towards the imported namespace.  */
+             sym = cp_lookup_symbol_via_imports (current->import_src,
+                                                 name, block,
+                                                 domain, 1, 0, 0);
+           }
+
+         if (sym.symbol != NULL)
+           return sym;
        }
     }
 
-  return NULL;
+  return null_block_symbol;
 }
 
-/* Helper function that searches an array of symbols for one named
-   NAME.  */
+/* Helper function that searches an array of symbols for one named NAME.  */
 
 static struct symbol *
 search_symbol_list (const char *name, int num,
@@ -420,36 +493,50 @@ search_symbol_list (const char *name, int num,
   return NULL;
 }
 
-/* Like cp_lookup_symbol_imports, but if BLOCK is a function, it
+/* Like cp_lookup_symbol_via_imports, but if BLOCK is a function, it
    searches through the template parameters of the function and the
    function's type.  */
 
-struct symbol *
+struct block_symbol
 cp_lookup_symbol_imports_or_template (const char *scope,
                                      const char *name,
                                      const struct block *block,
                                      const domain_enum domain)
 {
   struct symbol *function = BLOCK_FUNCTION (block);
+  struct block_symbol result;
 
-  if (function != NULL && SYMBOL_LANGUAGE (function) == language_cplus)
+  if (symbol_lookup_debug)
     {
-      int i;
-      struct cplus_specific *cps
-       = function->ginfo.language_specific.cplus_specific;
+      fprintf_unfiltered (gdb_stdlog,
+                         "cp_lookup_symbol_imports_or_template"
+                         " (%s, %s, %s, %s)\n",
+                         scope, name, host_address_to_string (block),
+                         domain_name (domain));
+    }
 
+  if (function != NULL && SYMBOL_LANGUAGE (function) == language_cplus)
+    {
       /* Search the function's template parameters.  */
       if (SYMBOL_IS_CPLUS_TEMPLATE_FUNCTION (function))
        {
-         struct template_symbol *templ 
+         struct template_symbol *templ
            = (struct template_symbol *) function;
-         struct symbol *result;
+         struct symbol *sym = search_symbol_list (name,
+                                                  templ->n_template_arguments,
+                                                  templ->template_arguments);
 
-         result = search_symbol_list (name,
-                                      templ->n_template_arguments,
-                                      templ->template_arguments);
-         if (result != NULL)
-           return result;
+         if (sym != NULL)
+           {
+             if (symbol_lookup_debug)
+               {
+                 fprintf_unfiltered (gdb_stdlog,
+                                     "cp_lookup_symbol_imports_or_template"
+                                     " (...) = %s\n",
+                                     host_address_to_string (sym));
+               }
+             return (struct block_symbol) {sym, block};
+           }
        }
 
       /* Search the template parameters of the function's defining
@@ -457,78 +544,119 @@ cp_lookup_symbol_imports_or_template (const char *scope,
       if (SYMBOL_NATURAL_NAME (function))
        {
          struct type *context;
-         char *name_copy = xstrdup (SYMBOL_NATURAL_NAME (function));
-         struct cleanup *cleanups = make_cleanup (xfree, name_copy);
+         std::string name_copy (SYMBOL_NATURAL_NAME (function));
          const struct language_defn *lang = language_def (language_cplus);
-         struct gdbarch *arch = SYMBOL_SYMTAB (function)->objfile->gdbarch;
+         struct gdbarch *arch = symbol_arch (function);
          const struct block *parent = BLOCK_SUPERBLOCK (block);
+         struct symbol *sym;
 
          while (1)
            {
-             struct symbol *result;
-             unsigned int prefix_len = cp_entire_prefix_len (name_copy);
+             unsigned int prefix_len
+               = cp_entire_prefix_len (name_copy.c_str ());
 
              if (prefix_len == 0)
                context = NULL;
              else
                {
-                 name_copy[prefix_len] = '\0';
+                 name_copy.erase (prefix_len);
                  context = lookup_typename (lang, arch,
-                                            name_copy,
+                                            name_copy.c_str (),
                                             parent, 1);
                }
 
              if (context == NULL)
                break;
 
-             result
+             sym
                = search_symbol_list (name,
                                      TYPE_N_TEMPLATE_ARGUMENTS (context),
                                      TYPE_TEMPLATE_ARGUMENTS (context));
-             if (result != NULL)
-               return result;
+             if (sym != NULL)
+               {
+                 if (symbol_lookup_debug)
+                   {
+                     fprintf_unfiltered
+                       (gdb_stdlog,
+                        "cp_lookup_symbol_imports_or_template (...) = %s\n",
+                        host_address_to_string (sym));
+                   }
+                 return (struct block_symbol) {sym, parent};
+               }
            }
-
-         do_cleanups (cleanups);
        }
     }
 
-  return cp_lookup_symbol_imports (scope, name, block, domain, 1, 1);
+  result = cp_lookup_symbol_via_imports (scope, name, block, domain, 0, 1, 1);
+  if (symbol_lookup_debug)
+    {
+      fprintf_unfiltered (gdb_stdlog,
+                         "cp_lookup_symbol_imports_or_template (...) = %s\n",
+                         result.symbol != NULL
+                         ? host_address_to_string (result.symbol) : "NULL");
+    }
+  return result;
 }
 
- /* Searches for NAME in the current namespace, and by applying
-    relevant import statements belonging to BLOCK and its parents.
-    SCOPE is the namespace scope of the context in which the search is
-    being evaluated.  */
+/* Search for NAME by applying relevant import statements belonging to BLOCK
+   and its parents.  SCOPE is the namespace scope of the context in which the
+   search is being evaluated.  */
 
-struct symbol*
+static struct block_symbol
+cp_lookup_symbol_via_all_imports (const char *scope, const char *name,
+                                 const struct block *block,
+                                 const domain_enum domain)
+{
+  struct block_symbol sym;
+
+  while (block != NULL)
+    {
+      sym = cp_lookup_symbol_via_imports (scope, name, block, domain, 0, 0, 1);
+      if (sym.symbol)
+       return sym;
+
+      block = BLOCK_SUPERBLOCK (block);
+    }
+
+  return null_block_symbol;
+}
+
+/* Searches for NAME in the current namespace, and by applying
+   relevant import statements belonging to BLOCK and its parents.
+   SCOPE is the namespace scope of the context in which the search is
+   being evaluated.  */
+
+struct block_symbol
 cp_lookup_symbol_namespace (const char *scope,
                             const char *name,
                             const struct block *block,
                             const domain_enum domain)
 {
-  struct symbol *sym;
-  
-  /* First, try to find the symbol in the given namespace.  */
-  sym = cp_lookup_symbol_in_namespace (scope, name,
-                                      block, domain);
-  if (sym != NULL)
-    return sym;
+  struct block_symbol sym;
 
-  /* Search for name in namespaces imported to this and parent
-     blocks.  */
-  while (block != NULL)
+  if (symbol_lookup_debug)
     {
-      sym = cp_lookup_symbol_imports (scope, name, block,
-                                     domain, 0, 1);
+      fprintf_unfiltered (gdb_stdlog,
+                         "cp_lookup_symbol_namespace (%s, %s, %s, %s)\n",
+                         scope, name, host_address_to_string (block),
+                         domain_name (domain));
+    }
 
-      if (sym)
-       return sym;
+  /* First, try to find the symbol in the given namespace.  */
+  sym = cp_lookup_symbol_in_namespace (scope, name, block, domain, 1);
 
-      block = BLOCK_SUPERBLOCK (block);
-    }
+  /* Search for name in namespaces imported to this and parent blocks.  */
+  if (sym.symbol == NULL)
+    sym = cp_lookup_symbol_via_all_imports (scope, name, block, domain);
 
-  return NULL;
+  if (symbol_lookup_debug)
+    {
+      fprintf_unfiltered (gdb_stdlog,
+                         "cp_lookup_symbol_namespace (...) = %s\n",
+                         sym.symbol != NULL
+                           ? host_address_to_string (sym.symbol) : "NULL");
+    }
+  return sym;
 }
 
 /* Lookup NAME at namespace scope (or, in C terms, in static and
@@ -546,20 +674,21 @@ cp_lookup_symbol_namespace (const char *scope,
    "A::x", and if that call fails, then the first call looks for
    "x".  */
 
-static struct symbol *
-lookup_namespace_scope (const char *name,
+static struct block_symbol
+lookup_namespace_scope (const struct language_defn *langdef,
+                       const char *name,
                        const struct block *block,
                        const domain_enum domain,
                        const char *scope,
                        int scope_len)
 {
-  char *namespace;
+  char *the_namespace;
 
   if (scope[scope_len] != '\0')
     {
       /* Recursively search for names in child namespaces first.  */
 
-      struct symbol *sym;
+      struct block_symbol sym;
       int new_scope_len = scope_len;
 
       /* If the current scope is followed by "::", skip past that.  */
@@ -569,109 +698,289 @@ lookup_namespace_scope (const char *name,
          new_scope_len += 2;
        }
       new_scope_len += cp_find_first_component (scope + new_scope_len);
-      sym = lookup_namespace_scope (name, block, domain,
+      sym = lookup_namespace_scope (langdef, name, block, domain,
                                    scope, new_scope_len);
-      if (sym != NULL)
+      if (sym.symbol != NULL)
        return sym;
     }
 
   /* Okay, we didn't find a match in our children, so look for the
-     name in the current namespace.  */
-
-  namespace = alloca (scope_len + 1);
-  strncpy (namespace, scope, scope_len);
-  namespace[scope_len] = '\0';
-  return cp_lookup_symbol_in_namespace (namespace, name,
-                                       block, domain);
+     name in the current namespace.
+
+     If we there is no scope and we know we have a bare symbol, then short
+     circuit everything and call cp_lookup_bare_symbol directly.
+     This isn't an optimization, rather it allows us to pass LANGDEF which
+     is needed for primitive type lookup.  The test doesn't have to be
+     perfect: if NAME is a bare symbol that our test doesn't catch (e.g., a
+     template symbol with "::" in the argument list) then
+     cp_lookup_symbol_in_namespace will catch it.  */
+
+  if (scope_len == 0 && strchr (name, ':') == NULL)
+    return cp_lookup_bare_symbol (langdef, name, block, domain, 1);
+
+  the_namespace = (char *) alloca (scope_len + 1);
+  strncpy (the_namespace, scope, scope_len);
+  the_namespace[scope_len] = '\0';
+  return cp_lookup_symbol_in_namespace (the_namespace, name,
+                                       block, domain, 1);
 }
 
-/* Look up NAME in BLOCK's static block and in global blocks.  If
-   ANONYMOUS_NAMESPACE is nonzero, the symbol in question is located
-   within an anonymous namespace.  Other arguments are as in
-   cp_lookup_symbol_nonlocal.  */
+/* The C++-specific version of name lookup for static and global
+   names.  This makes sure that names get looked for in all namespaces
+   that are in scope.  NAME is the natural name of the symbol that
+   we're looking for, BLOCK is the block that we're searching within,
+   DOMAIN says what kind of symbols we're looking for.  */
 
-static struct symbol *
-lookup_symbol_file (const char *name,
-                   const struct block *block,
-                   const domain_enum domain,
-                   int anonymous_namespace)
+struct block_symbol
+cp_lookup_symbol_nonlocal (const struct language_defn *langdef,
+                          const char *name,
+                          const struct block *block,
+                          const domain_enum domain)
 {
-  struct symbol *sym = NULL;
+  struct block_symbol sym;
+  const char *scope = block_scope (block);
 
-  sym = lookup_symbol_static (name, block, domain);
-  if (sym != NULL)
-    return sym;
+  if (symbol_lookup_debug)
+    {
+      fprintf_unfiltered (gdb_stdlog,
+                         "cp_lookup_symbol_non_local"
+                         " (%s, %s (scope %s), %s)\n",
+                         name, host_address_to_string (block), scope,
+                         domain_name (domain));
+    }
 
-  if (anonymous_namespace)
+  /* First, try to find the symbol in the given namespace, and all
+     containing namespaces.  */
+  sym = lookup_namespace_scope (langdef, name, block, domain, scope, 0);
+
+  /* Search for name in namespaces imported to this and parent blocks.  */
+  if (sym.symbol == NULL)
+    sym = cp_lookup_symbol_via_all_imports (scope, name, block, domain);
+
+  if (symbol_lookup_debug)
     {
-      /* Symbols defined in anonymous namespaces have external linkage
-        but should be treated as local to a single file nonetheless.
-        So we only search the current file's global block.  */
+      fprintf_unfiltered (gdb_stdlog,
+                         "cp_lookup_symbol_nonlocal (...) = %s\n",
+                         (sym.symbol != NULL
+                          ? host_address_to_string (sym.symbol)
+                          : "NULL"));
+    }
+  return sym;
+}
 
-      const struct block *global_block = block_global_block (block);
-      
-      if (global_block != NULL)
-       sym = lookup_symbol_aux_block (name, global_block, domain);
+/* Search through the base classes of PARENT_TYPE for a base class
+   named NAME and return its type.  If not found, return NULL.  */
+
+struct type *
+cp_find_type_baseclass_by_name (struct type *parent_type, const char *name)
+{
+  int i;
+
+  parent_type = check_typedef (parent_type);
+  for (i = 0; i < TYPE_N_BASECLASSES (parent_type); ++i)
+    {
+      struct type *type = check_typedef (TYPE_BASECLASS (parent_type, i));
+      const char *base_name = TYPE_BASECLASS_NAME (parent_type, i);
+
+      if (base_name == NULL)
+       continue;
+
+      if (streq (base_name, name))
+       return type;
+
+      type = cp_find_type_baseclass_by_name (type, name);
+      if (type != NULL)
+       return type;
     }
-  else
+
+  return NULL;
+}
+
+/* Search through the base classes of PARENT_TYPE for a symbol named
+   NAME in block BLOCK.  */
+
+static struct block_symbol
+find_symbol_in_baseclass (struct type *parent_type, const char *name,
+                         const struct block *block, const domain_enum domain,
+                         int is_in_anonymous)
+{
+  int i;
+  struct block_symbol sym;
+
+  sym.symbol = NULL;
+  sym.block = NULL;
+
+  for (i = 0; i < TYPE_N_BASECLASSES (parent_type); ++i)
     {
-      sym = lookup_symbol_global (name, block, domain);
+      struct type *base_type = TYPE_BASECLASS (parent_type, i);
+      const char *base_name = TYPE_BASECLASS_NAME (parent_type, i);
+
+      if (base_name == NULL)
+       continue;
+
+      std::string concatenated_name = std::string (base_name) + "::" + name;
+
+      sym = cp_lookup_nested_symbol_1 (base_type, name,
+                                      concatenated_name.c_str (),
+                                      block, domain, 1, is_in_anonymous);
+      if (sym.symbol != NULL)
+       break;
     }
 
   return sym;
 }
 
-/* Look up a type named NESTED_NAME that is nested inside the C++
-   class or namespace given by PARENT_TYPE, from within the context
-   given by BLOCK.  Return NULL if there is no such nested type.  */
+/* Helper function to look up NESTED_NAME in CONTAINER_TYPE and in DOMAIN
+   and within the context of BLOCK.
+   NESTED_NAME may have scope ("::").
+   CONTAINER_TYPE needn't have been "check_typedef'd" yet.
+   CONCATENATED_NAME is the fully scoped spelling of NESTED_NAME, it is
+   passed as an argument so that callers can control how space for it is
+   allocated.
+   If BASIC_LOOKUP is non-zero then perform a basic lookup of
+   CONCATENATED_NAME.  See cp_basic_lookup_symbol for details.
+   If IS_IN_ANONYMOUS is non-zero then CONCATENATED_NAME is in an anonymous
+   namespace.  */
+
+static struct block_symbol
+cp_lookup_nested_symbol_1 (struct type *container_type,
+                          const char *nested_name,
+                          const char *concatenated_name,
+                          const struct block *block,
+                          const domain_enum domain,
+                          int basic_lookup, int is_in_anonymous)
+{
+  struct block_symbol sym;
 
-struct type *
-cp_lookup_nested_type (struct type *parent_type,
-                      const char *nested_name,
-                      const struct block *block)
+  /* NOTE: carlton/2003-11-10: We don't treat C++ class members
+     of classes like, say, data or function members.  Instead,
+     they're just represented by symbols whose names are
+     qualified by the name of the surrounding class.  This is
+     just like members of namespaces; in particular,
+     cp_basic_lookup_symbol works when looking them up.  */
+
+  if (basic_lookup)
+    {
+      sym = cp_basic_lookup_symbol (concatenated_name, block, domain,
+                                   is_in_anonymous);
+      if (sym.symbol != NULL)
+       return sym;
+    }
+
+  /* Now search all static file-level symbols.  We have to do this for things
+     like typedefs in the class.  We do not try to guess any imported
+     namespace as even the fully specified namespace search is already not
+     C++ compliant and more assumptions could make it too magic.  */
+
+  /* First search in this symtab, what we want is possibly there.  */
+  sym = lookup_symbol_in_static_block (concatenated_name, block, domain);
+  if (sym.symbol != NULL)
+    return sym;
+
+  /* Nope.  We now have to search all static blocks in all objfiles,
+     even if block != NULL, because there's no guarantees as to which
+     symtab the symbol we want is in.  Except for symbols defined in
+     anonymous namespaces should be treated as local to a single file,
+     which we just searched.  */
+  if (!is_in_anonymous)
+    {
+      sym = lookup_static_symbol (concatenated_name, domain);
+      if (sym.symbol != NULL)
+       return sym;
+    }
+
+  /* If this is a class with baseclasses, search them next.  */
+  container_type = check_typedef (container_type);
+  if (TYPE_N_BASECLASSES (container_type) > 0)
+    {
+      sym = find_symbol_in_baseclass (container_type, nested_name, block,
+                                     domain, is_in_anonymous);
+      if (sym.symbol != NULL)
+       return sym;
+    }
+
+  return null_block_symbol;
+}
+
+/* Look up a symbol named NESTED_NAME that is nested inside the C++
+   class or namespace given by PARENT_TYPE, from within the context
+   given by BLOCK, and in DOMAIN.
+   Return NULL if there is no such nested symbol.  */
+
+struct block_symbol
+cp_lookup_nested_symbol (struct type *parent_type,
+                        const char *nested_name,
+                        const struct block *block,
+                        const domain_enum domain)
 {
+  /* type_name_no_tag_or_error provides better error reporting using the
+     original type.  */
+  struct type *saved_parent_type = parent_type;
+
+  parent_type = check_typedef (parent_type);
+
+  if (symbol_lookup_debug)
+    {
+      const char *type_name = type_name_no_tag (saved_parent_type);
+
+      fprintf_unfiltered (gdb_stdlog,
+                         "cp_lookup_nested_symbol (%s, %s, %s, %s)\n",
+                         type_name != NULL ? type_name : "unnamed",
+                         nested_name, host_address_to_string (block),
+                         domain_name (domain));
+    }
+
   switch (TYPE_CODE (parent_type))
     {
     case TYPE_CODE_STRUCT:
     case TYPE_CODE_NAMESPACE:
     case TYPE_CODE_UNION:
+    case TYPE_CODE_ENUM:
+    /* NOTE: Handle modules here as well, because Fortran is re-using the C++
+       specific code to lookup nested symbols in modules, by calling the
+       function pointer la_lookup_symbol_nonlocal, which ends up here.  */
+    case TYPE_CODE_MODULE:
       {
-       /* NOTE: carlton/2003-11-10: We don't treat C++ class members
-          of classes like, say, data or function members.  Instead,
-          they're just represented by symbols whose names are
-          qualified by the name of the surrounding class.  This is
-          just like members of namespaces; in particular,
-          lookup_symbol_namespace works when looking them up.  */
-
-       const char *parent_name = TYPE_TAG_NAME (parent_type);
-       struct symbol *sym
-         = cp_lookup_symbol_in_namespace (parent_name, nested_name,
-                                          block, VAR_DOMAIN);
+       int size;
+       const char *parent_name = type_name_no_tag_or_error (saved_parent_type);
+       struct block_symbol sym;
        char *concatenated_name;
+       int is_in_anonymous;
+
+       size = strlen (parent_name) + 2 + strlen (nested_name) + 1;
+       concatenated_name = (char *) alloca (size);
+       xsnprintf (concatenated_name, size, "%s::%s",
+                  parent_name, nested_name);
+       is_in_anonymous = cp_is_in_anonymous (concatenated_name);
 
-       if (sym != NULL && SYMBOL_CLASS (sym) == LOC_TYPEDEF)
-         return SYMBOL_TYPE (sym);
-
-       /* Now search all static file-level symbols.  Not strictly
-          correct, but more useful than an error.  We do not try to
-          guess any imported namespace as even the fully specified
-          namespace seach is is already not C++ compliant and more
-          assumptions could make it too magic.  */
-
-       concatenated_name = alloca (strlen (parent_name) + 2
-                                   + strlen (nested_name) + 1);
-       sprintf (concatenated_name, "%s::%s",
-                parent_name, nested_name);
-       sym = lookup_static_symbol_aux (concatenated_name,
-                                       VAR_DOMAIN);
-       if (sym != NULL && SYMBOL_CLASS (sym) == LOC_TYPEDEF)
-         return SYMBOL_TYPE (sym);
-
-       return NULL;
+       sym = cp_lookup_nested_symbol_1 (parent_type, nested_name,
+                                        concatenated_name, block, domain,
+                                        1, is_in_anonymous);
+
+       if (symbol_lookup_debug)
+         {
+           fprintf_unfiltered (gdb_stdlog,
+                               "cp_lookup_nested_symbol (...) = %s\n",
+                               (sym.symbol != NULL
+                                ? host_address_to_string (sym.symbol)
+                                : "NULL"));
+         }
+       return sym;
       }
+
+    case TYPE_CODE_FUNC:
+    case TYPE_CODE_METHOD:
+      if (symbol_lookup_debug)
+       {
+         fprintf_unfiltered (gdb_stdlog,
+                             "cp_lookup_nested_symbol (...) = NULL"
+                             " (func/method)\n");
+       }
+      return null_block_symbol;
+
     default:
       internal_error (__FILE__, __LINE__,
-                     _("cp_lookup_nested_type called "
+                     _("cp_lookup_nested_symbol called "
                        "on a non-aggregate type."));
     }
 }
@@ -738,9 +1047,9 @@ cp_lookup_transparent_type_loop (const char *name,
        return retval;
     }
 
-  full_name = alloca (scope_length + 2 + strlen (name) + 1);
+  full_name = (char *) alloca (scope_length + 2 + strlen (name) + 1);
   strncpy (full_name, scope, scope_length);
-  strncpy (full_name + scope_length, "::", 2);
+  memcpy (full_name + scope_length, "::", 2);
   strcpy (full_name + scope_length + 2, name);
 
   return basic_lookup_transparent_type (full_name);
@@ -750,14 +1059,11 @@ cp_lookup_transparent_type_loop (const char *name,
    obsolete.  */
 
 static void
-maintenance_cplus_namespace (char *args, int from_tty)
+maintenance_cplus_namespace (const char *args, int from_tty)
 {
   printf_unfiltered (_("The `maint namespace' command was removed.\n"));
 }
 
-/* Provide a prototype to silence -Wmissing-prototypes.  */
-extern initialize_file_ftype _initialize_cp_namespace;
-
 void
 _initialize_cp_namespace (void)
 {