[PATCH] D150637: [lld][ELF] Add option for suppressing section type mismatch warnings

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 21:11:48 PDT 2023


MaskRay added a comment.

In D150637#4345199 <https://reviews.llvm.org/D150637#4345199>, @peter.smith wrote:

> ...
>
> To summarise:
>
> - I would prefer we not put out the warning with NOLOAD as default as I think it is working as designed.
> - An option to suppress/enable the warning is a good second choice. I'd prefer us to suppress by default, particularly for NOLOAD.

Thanks for the write-up. I prefer that we don't give a warning with `NOLOAD`.
For 2, I prefer that we don't have an option to toggle the warning.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150637



More information about the llvm-commits mailing list