[PATCH] D113534: [lld-macho] Do not warn on install_name for bundles

Vincent Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 9 18:45:30 PST 2021


thevinster added a comment.

> Maybe we should instead have a flag to disable the warning?

The reason for this change is that even though it's a no-op, it's polluting quite bunch of internal build logs which make it hard to uncover what warnings are actually problems and what aren't. As far as the flag to disable the warning, would it better to mirror ld64's behavior and have a flag to enable this check instead? (Alternatively, we could also make the flag to disable the warning a default as well). wdyt?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113534



More information about the llvm-commits mailing list