2014-03-13 21:41 GMT+04:00 Georg-Johann Lay <a...@gjlay.de>: > 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.
I'm prefer `targetm.strip_name_encoding' or `default_strip_name_encoding'. May be `default_strip_name_encoding' is better because it's used in few ports. Denis.
