Am 03/13/2014 04:41 PM, schrieb Senthil Kumar Selvaraj:
On Thu, Mar 13, 2014 at 02:24:06PM +0100, Georg-Johann Lay wrote:
Problem is that the assembler name might or might not be prefixed by '*'
depending on when TARGET_SET_CURRENT_FUNCTION is called.
The change is just to fix wrong warning because the current implementation
of TARGET_SET_CURRENT_FUNCTION /always/ skips the first char when the
assembler name is set.
FWIW, there's default_strip_name_encoding (varasm.c), which does the same
thing, and is used by a couple of other targets.
Yes, I know.
But I would prefer targetm.strip_name_encoding then, even though avr does not
implement it.
Let Denis decide what he prefers.
Johann