[PATCH] D41046: [ELF] Make overlapping output sections an error

[Alexander Richardson via Phabricator via llvm-commits]

arichardson added a comment.

In https://reviews.llvm.org/D41046#950879, @grimar wrote:

> In https://reviews.llvm.org/D41046#950367, @arichardson wrote:
>
> > Speed up lld invocation in test/ELF/many-alloc-sections.s lld invocation from 290s to <1s by not checking empty sections
>
>
> I think we already had patches for that purpose before (I can be missing details, though I mean general idea was to avoid overlapping things when linker script is used, 
>  probably it were PT_LOADs, but anyways).
>  And I think last time we stopped on a point that we are not going to stop users from breaking output if they want to do that, because with linker script it is easy to break everything.
>
> That position can be revisited probably, if you have good arguments for that :)


My argument is that I wasted hours of debugging time due to a typo in the linker script. If lld had warned me about the overlapping output sections that would not have happened.
If a user really wants to create broken output with a linker script we have `--noinhibit-exec` for that use case.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D41046