This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: Fix sim fallout from arm assembler complaining about symbols named as insns
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- To: vapier at gentoo dot org
- Cc: nickc at redhat dot com, gdb-patches at sourceware dot org, binutils at sourceware dot org
- Date: Fri, 8 May 2015 01:57:38 +0200
- Subject: Re: Fix sim fallout from arm assembler complaining about symbols named as insns
- Authentication-results: sourceware.org; auth=none
> From: Mike Frysinger <vapier@gentoo.org>
> Date: Thu, 7 May 2015 08:36:14 +0200
> so people can't have a global variable named "bl" now ? or "ldr" ? that
> doesn't seem like the right direction in which case this patch isn't really
> needed ...
> -mike
Yeah, as you've noticed I haven't applied it yet.
I reconsidered after VP noting that there actually *are* gcc
test-suite cases failing because of this.
Sorry Nick, but I'd like to see a -mwarn-syms being passed by
default by gcc, before believing this warning-by-default
behavior is here to stay. The cure (helping a rare programming
error with an unexpected failure mode by means of a sledgehammer
fix) is IMHO worse than the disease.
Add a target hook to the SYM = EXPR gas code and check there?
(But don't forget to check that the symbol doesn't yet exist or
at least isn't global.)
brgds, H-P