Re: Fix sim fallout from arm assembler complaining about symbols named as insns

Hans-Peter Nilsson
2015-05-07T23:57:45Z

> 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