#2288 asm naming bug

[demyan]

bug, Error: multiple definitions error

sample:

void func() { }

void func_start() { }

void func_end() { }

[demyan]

target: z80

[Maarten Brock]

I have often wondered what these generated symbols are for. I can only think of viewing the function size in the memory map. Then they should at least not clash with the generated symbols from your C code. But since I don't think any other target uses them they can also be removed.

[Kio]

I'd vote for removing them too.
I'm currently updating my Z80 assembler to handle the sdcc output and i don't see any real use for them. They just bloat the labels list. And they're global.

[Philipp Klaus Krause]

No one showed up sating that the _start and _end things are used, so I'll remove them even though they seem to have been an intentional feature once.

Philipp

[Philipp Klaus Krause]

Fixed in revision #9195.

Philipp

[Brian Ruthven]

Rats, I was too late here. I do use these symbols (although only in the offline debugger, not the Z80 code itself. Is there any chance we can have them back disabled by default, but with a flag to enable them? That would hopefully satisfy everybody...

[Brian Ruthven]

Alternatively, a size sent to the symbol table would probably be just as useful. I'll raise a Feature Request.