[PATCH] D155989: [LLD][ELF] Warn on invalid local symbols

[Vincent Lee via Phabricator via llvm-commits]

thevinster added a comment.

> I wonder whether you can consider binary patching the broken shared objects. I have done similar things for my employer (for symbol versioning, not for this STB_LOCAL instance) as well..
> If binary patching doesn't look great, use --noinhibit-exec.

Patching the binary is something we're probably going to try and pursue, but I wanted to get your thoughts first whether you feel strongly against switching this back to a warning. Using `--noinhibit-exec` relaxes other errors as well and would prefer not to do that to avoid more broken shared objects from creeping. So far, it's just this specific message that I've seen so far.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155989/new/

https://reviews.llvm.org/D155989