diff options
| author | Patrick Palka <patrick@parcs.ath.cx> | 2015-06-29 10:45:47 -0400 |
|---|---|---|
| committer | Patrick Palka <patrick@parcs.ath.cx> | 2015-08-29 08:53:08 -0400 |
| commit | aa78b3b28aeff4bb9977a313f5a8002d920b34c5 (patch) | |
| tree | 9929b889caa9329b4a0f7665c26e252eaaa18702 /gdb/gdbarch.c | |
| parent | 8f57eec2fb31db5ab4598f09136e7978b7f84c97 (diff) | |
| download | gdb-aa78b3b28aeff4bb9977a313f5a8002d920b34c5.zip gdb-aa78b3b28aeff4bb9977a313f5a8002d920b34c5.tar.gz gdb-aa78b3b28aeff4bb9977a313f5a8002d920b34c5.tar.bz2 | |
Use gdbarch obstack to allocate the TYPE_NAME string in arch_type
Since the type whose name is being set is now being allocated on the
gdbarch obstack, we should allocate its TYPE_NAME on the obstack too.
This reduces the number of individual valgrind warnings for the command
"gdb gdb" from ~300 to ~150.
Tested on x86_64-unknown-linux-gnu.
gdb/ChangeLog:
* gdbarch.h (gdbarch_obstack_strdup): Declare.
* gdbarch.c (gdbarch_obstack_strdup): Define.
* gdbtypes.c (arch_type): Use it.
Diffstat (limited to 'gdb/gdbarch.c')
| -rw-r--r-- | gdb/gdbarch.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/gdb/gdbarch.c b/gdb/gdbarch.c index 0d4142b..37ce22a 100644 --- a/gdb/gdbarch.c +++ b/gdb/gdbarch.c @@ -449,6 +449,16 @@ gdbarch_obstack_zalloc (struct gdbarch *arch, long size) return data; } +/* See gdbarch.h. */ + +char * +gdbarch_obstack_strdup (struct gdbarch *gdbarch, const char *string) +{ + char *obstring = gdbarch_obstack_zalloc (gdbarch, strlen (string) + 1); + strcpy (obstring, string); + return obstring; +} + /* Free a gdbarch struct. This should never happen in normal operation --- once you've created a gdbarch, you keep it around. |