Use TOLOWER in SYMBOL_HASH_NEXT
The support for setting breakpoint in functions with ABI tags patch will add a use of SYMBOL_HASH_NEXT in cp-support.c, which fails to compile with: src/gdb/cp-support.c:38:0: src/gdb/cp-support.c: In function ‘unsigned int cp_search_name_hash(const char*)’: src/gdb/../include/safe-ctype.h:148:20: error: ‘do_not_use_tolower_with_safe_ctype’ was not declared in this scope #define tolower(c) do_not_use_tolower_with_safe_ctype ^ src/gdb/minsyms.h:174:18: note: in expansion of macro ‘tolower’ ((hash) * 67 + tolower ((unsigned char) (c)) - 113) ^ src/gdb/cp-support.c:1677:14: note: in expansion of macro ‘SYMBOL_HASH_NEXT’ hash = SYMBOL_HASH_NEXT (hash, *string); ^ This fixes the problem before it happens. I was somewhat worried about whether this might have an impact with languages that are case insensitive, but I convinced myself that it doesn't. As bonus, this improves GDB's minsym interning performance a bit (3%-10%). See <https://sourceware.org/ml/gdb/2017-11/msg00021.html>. gdb/ChangeLog: 2017-11-25 Pedro Alves <palves@redhat.com> * dictionary.c: Include "safe-ctype.h". * minsyms.c: Include "safe-ctype.h". * minsyms.c (SYMBOL_HASH_NEXT): Use TOLOWER instead of tolower.
This commit is contained in:
parent
a81aaca057
commit
deeeba559b
|
@ -1,3 +1,9 @@
|
|||
2017-11-25 Pedro Alves <palves@redhat.com>
|
||||
|
||||
* dictionary.c: Include "safe-ctype.h".
|
||||
* minsyms.c: Include "safe-ctype.h".
|
||||
* minsyms.c (SYMBOL_HASH_NEXT): Use TOLOWER instead of tolower.
|
||||
|
||||
2017-11-25 Pedro Alves <palves@redhat.com>
|
||||
|
||||
* completer.c (complete_line_internal_1): Skip spaces until the
|
||||
|
|
|
@ -26,6 +26,7 @@
|
|||
#include "symtab.h"
|
||||
#include "buildsym.h"
|
||||
#include "dictionary.h"
|
||||
#include "safe-ctype.h"
|
||||
|
||||
/* This file implements dictionaries, which are tables that associate
|
||||
symbols to names. They are represented by an opaque type 'struct
|
||||
|
|
|
@ -52,6 +52,7 @@
|
|||
#include "cli/cli-utils.h"
|
||||
#include "symbol.h"
|
||||
#include <algorithm>
|
||||
#include "safe-ctype.h"
|
||||
|
||||
/* See minsyms.h. */
|
||||
|
||||
|
|
|
@ -177,7 +177,7 @@ unsigned int msymbol_hash_iw (const char *);
|
|||
requirements. */
|
||||
|
||||
#define SYMBOL_HASH_NEXT(hash, c) \
|
||||
((hash) * 67 + tolower ((unsigned char) (c)) - 113)
|
||||
((hash) * 67 + TOLOWER ((unsigned char) (c)) - 113)
|
||||
|
||||
|
||||
|
||||
|
|
Loading…
Reference in New Issue