[PATCH] D113071: [lld-macho] Ignore -arch_errors_fatal since it's enabled by default

Keith Smiley via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 2 20:16:16 PDT 2021


keith added a comment.

In D113071#3104899 <https://reviews.llvm.org/D113071#3104899>, @int3 wrote:

> Right, so ld64 doesn't have a way to disable this `fatal` because it's already disabled...

Exactly. I was just saying that we could safely ignore this (if we were ok to have the behavior difference) that we wouldn't need to publicize this flag at all.

> But yeah I would prefer that we hewed to ld64's behavior here so that LLD is easier to use as a drop-in replacement

Ok so you think we should swap the existing error for a warning, and enable this flag to act as it does today?

FWIW I think the risk of this being a blocker for adoption is pretty low, but I totally take your point.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113071



More information about the llvm-commits mailing list