[PATCH] D38239: [ELF] - Define linkerscript symbols early.

[George Rimar via llvm-commits]

>> Though currently with Out::ElfHeader following cast asserts for me, because Out::ElfHeader is an input section,
>> not output: https://github.com/llvm-mirror/lld/blob/master/ELF/SymbolTable.cpp#L441
>> Is section needed there for something unobvious ?
>
>That is the thin lto case, no? That is where I think there is something
>odd with thin lto. It is producing a symbol we didn't ask for. Since the
>symbol is already defined, we end up trying to report a duplicated
>symbol error and crashing.
>
>If thin lto would omit that symbol like regular LTO does we would not
>get any errors.
>
>Cheers,
>Rafael

Ok. Want me to try investigate what is wrong with thin lto case then ?
(I can try, though that probably be after 8th jan, we have a week+1 day of NY holidays starting
on next week here).

George.