From: Doug Evans <xdje42@gmail.com>
Date: Fri, 7 Nov 2014 07:34:28 +0000 (-0800)
Subject: symtab.c (basic_lookup_symbol_nonlocal): Add comment.
X-Git-Tag: hjl/linux/release/2.25.51.0.1~3^2~44^2~9
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d9060ba60d1523c87e44eaeb7706d168089936a3;p=thirdparty%2Fbinutils-gdb.git

symtab.c (basic_lookup_symbol_nonlocal): Add comment.

gdb/ChangeLog:

	* symtab.c (basic_lookup_symbol_nonlocal): Add comment.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 5500c15cd8b..882262d7390 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2014-11-06  Doug Evans  <xdje42@gmail.com>
+
+	* symtab.c (basic_lookup_symbol_nonlocal): Add comment.
+
 2014-11-06  Doug Evans  <xdje42@gmail.com>
 
 	* symtab.c (lookup_local_symbol): Renamed from lookup_symbol_aux_local.
diff --git a/gdb/symtab.c b/gdb/symtab.c
index 6f483bf3d1b..e2a1e56f660 100644
--- a/gdb/symtab.c
+++ b/gdb/symtab.c
@@ -1792,6 +1792,10 @@ basic_lookup_symbol_nonlocal (const char *name,
      than that one, so I don't think we should worry about that for
      now.  */
 
+  /* NOTE: dje/2014-10-26: The lookup in all objfiles search could skip
+     the current objfile.  Searching the current objfile first is useful
+     for both matching user expectations as well as performance.  */
+
   sym = lookup_symbol_in_static_block (name, block, domain);
   if (sym != NULL)
     return sym;