[PATCH] D144471: [ELF] Prohibit icf=safe warning from being fatal

Marco Elver via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 11:17:20 PST 2023


melver abandoned this revision.
melver added a comment.

In D144471#4142299 <https://reviews.llvm.org/D144471#4142299>, @MaskRay wrote:

> I don't think we want more diagnostic levels. The warning is not fatal, it's just that some users use `--fatal-warnings` to upgrade warnings to errors.
> If this warning is annoying under `--fatal-warnings`, we can think of `message(...)`. But right now, it seems appropriate for the user to be aware of the problem and discard `.llvm_addrsig` by themselves.
> (I have more notes in https://maskray.me/blog/2020-11-15-explain-gnu-linker-options )

Thanks - I'll abandon this for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144471



More information about the llvm-commits mailing list