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

Vincent Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 21 13:45:24 PDT 2023


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



More information about the llvm-commits mailing list