Re: [Tinycc-devel] forward asm symbols vs static

[Michael Matz]

Hi,

On Sun, 19 Nov 2017, Michael Matz wrote:

> > Aha!  Indeed, p3 should be STB_LOCAL, and now (after my revert or before
> > your patch) isn't anymore.  Okay, I'll work on this.  Need to find a way
> > to get both things working, hmm :)
>
> Fixed in mob.

Gnah, what a rats hole :-/ I found another failure mode that occurs only 
when defaulting asm labels to VT_STATIC/STB_LOCAL.  I don't yet want to 
revert as it fixes demonstratable bugs, but OTOH I lack time tonight to 
fix this third problem as well.  If you want to release soon, I'd suggest 
to revert d0db2175 back to a8ece0f2 (i.e. to a state where the 
alloca86_64.S/p3 reoccurs, like it always had in the past).  If you can 
wait some more days with the release I'm going to fix it properly 
(hopefully).

(FWIW the problem occurs when building multiple files into one executable 
(it works when compiling the files individually and then linking the .o 
files together) where the files each contain multiple asm snippets 
containing forward references that eventually turn out to be global. 
local symbols aren't entered into the ELF hash tables and when they are 
globalized later are missing from them, ultimately leading to linker 
errors.  At least I think that's the reason :) This is all made a bit 
messy by the fact that the asm symtable lives only during one asm snippet, 
which is itself messy to fix because it shares the namespace with the C 
labels.)


Ciao,
Michael.