[PATCH] D30458: [ELF] - Make Bss and BssRelRo sections to be synthetic by nature.

[Rafael Avila de Espindola via llvm-commits]

Rui Ueyama via Phabricator <reviews at reviews.llvm.org> writes:

> ruiu added a comment.
>
> It doesn't seem like an improvement to me. Out::Bss and Out::BssRelRo are containers of synthetic sections, thus they are output sections. You could represent them as input sections, and we want to do if that simplifies code/design, but this patch doesn't seem to simplify it.

They are synthetic sections in the sense that a linker script can
put them in another output section:

  aabb : { *(.bss.*) }

Maybe we should revisit the idea of having one section per copy
relocation? We can do that if we add an offset to SharedSymbol or if we
replace the SharedSymbol with a DefinedRegular when a copy relocation is
created (I think I like the second option better).

Cheers,
Rafael