initializeSymbols currently sets Defined::section and handles non-prevailing
COMDAT groups. Move the code to the parallel postParse to reduce work from the
single-threading code path and make parallel section initialization infeasible.
Postpone reporting duplicate symbol errors so that the messages have the
section information. (Defined::section is assigned in postParse and another
thread may not have the information).
- duplicated-synthetic-sym.s: BinaryFile duplicate definition (very rare) now has no section information
- comdat-binding: %t/w.o %t/g.o leads to an undesired undefined symbol. This is not ideal but we report a diagnostic to inform that this is unsupported. (See release note)
- comdat-discarded-lazy.s: %tdef.o is unextracted. The new behavior (discarded section error) makes more sense
Depends on D120640