Remove unused variables from gdbserver
This removes a few unused variables from gdbserver. The x86-tdesc.h change is a bit unusual for this series. This file was not defining the multiple-include guard symbol, so I've added that here. Also, it is hard to determine when i386_expedite_regs will be needed, so this patch simply marks it ATTRIBUTE_UNUSED. gdb/gdbserver/ChangeLog 2018-07-22 Tom Tromey <tom@tromey.com> * win32-low.c (win32_create_inferior): Remove unused variables. * gdbreplay.c (remote_open): Remove unused variable. * remote-utils.c (remote_prepare): Remove unused variable. * x86-tdesc.h (X86_TDESC_H): Define. (amd64_expedite_regs): Define conditionally. (i386_expedite_regs): Mark ATTRIBUTE_UNUSED. * linux-x86-tdesc.c (i386_tdescs): Move inside #if. * remote-utils.c (readchar): Remove unused variable.
This commit is contained in:
parent
8c8807f439
commit
b0a7723d29
|
@ -1,3 +1,14 @@
|
|||
2018-07-22 Tom Tromey <tom@tromey.com>
|
||||
|
||||
* win32-low.c (win32_create_inferior): Remove unused variables.
|
||||
* gdbreplay.c (remote_open): Remove unused variable.
|
||||
* remote-utils.c (remote_prepare): Remove unused variable.
|
||||
* x86-tdesc.h (X86_TDESC_H): Define.
|
||||
(amd64_expedite_regs): Define conditionally.
|
||||
(i386_expedite_regs): Mark ATTRIBUTE_UNUSED.
|
||||
* linux-x86-tdesc.c (i386_tdescs): Move inside #if.
|
||||
* remote-utils.c (readchar): Remove unused variable.
|
||||
|
||||
2018-07-13 Pedro Alves <palves@redhat.com>
|
||||
|
||||
* linux-low.c (linux_kill): Change parameter to process_info
|
||||
|
|
|
@ -157,7 +157,6 @@ remote_open (char *name)
|
|||
#ifdef USE_WIN32API
|
||||
static int winsock_initialized;
|
||||
#endif
|
||||
char *port_str;
|
||||
int tmp;
|
||||
int tmp_desc;
|
||||
struct addrinfo hint;
|
||||
|
|
|
@ -69,10 +69,10 @@ xcr0_to_tdesc_idx (uint64_t xcr0, bool is_x32)
|
|||
return X86_TDESC_LAST;
|
||||
}
|
||||
|
||||
static struct target_desc *i386_tdescs[X86_TDESC_LAST] = { };
|
||||
|
||||
#if defined __i386__ || !defined IN_PROCESS_AGENT
|
||||
|
||||
static struct target_desc *i386_tdescs[X86_TDESC_LAST] = { };
|
||||
|
||||
/* Return the target description according to XCR0. */
|
||||
|
||||
const struct target_desc *
|
||||
|
|
|
@ -231,7 +231,6 @@ void
|
|||
remote_prepare (const char *name)
|
||||
{
|
||||
client_state &cs = get_client_state ();
|
||||
const char *port_str;
|
||||
#ifdef USE_WIN32API
|
||||
static int winsock_initialized;
|
||||
#endif
|
||||
|
@ -902,7 +901,6 @@ static unsigned char *readchar_bufp;
|
|||
static int
|
||||
readchar (void)
|
||||
{
|
||||
client_state &cs = get_client_state ();
|
||||
int ch;
|
||||
|
||||
if (readchar_bufcnt == 0)
|
||||
|
|
|
@ -634,8 +634,6 @@ win32_create_inferior (const char *program,
|
|||
#endif
|
||||
BOOL ret;
|
||||
DWORD flags;
|
||||
int argslen;
|
||||
int argc;
|
||||
PROCESS_INFORMATION pi;
|
||||
DWORD err;
|
||||
std::string str_program_args = stringify_argv (program_args);
|
||||
|
|
|
@ -16,11 +16,17 @@
|
|||
along with this program. If not, see <http://www.gnu.org/licenses/>. */
|
||||
|
||||
#ifndef X86_TDESC_H
|
||||
#define X86_TDESC_H
|
||||
|
||||
/* The "expedite" registers for x86 targets. */
|
||||
static const char *i386_expedite_regs[] = {"ebp", "esp", "eip", NULL};
|
||||
/* The "expedite" registers for x86 targets. Since whether the
|
||||
variable is used depends on host/configuration, we mark it
|
||||
ATTRIBUTE_UNUSED to keep it simple here. */
|
||||
static const char *i386_expedite_regs[] ATTRIBUTE_UNUSED
|
||||
= {"ebp", "esp", "eip", NULL};
|
||||
|
||||
#ifdef __x86_64__
|
||||
/* The "expedite" registers for x86_64 targets. */
|
||||
static const char *amd64_expedite_regs[] = {"rbp", "rsp", "rip", NULL};
|
||||
#endif
|
||||
|
||||
#endif /* X86_TDESC_H */
|
||||
|
|
Loading…
Reference in New Issue