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

Reply via email to