[PATCH] D113068: [lld-macho] Add -fatal-warnings alias

Keith Smiley via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 2 18:07:31 PDT 2021


keith added a comment.

In D113068#3104756 <https://reviews.llvm.org/D113068#3104756>, @MaskRay wrote:

> Options across binary formats are just so different. I think a user seeing `-option-name` may probably be more confused.

I did hide this by default in help to steer folks towards the default. For users who don't manually pass a lot of linker flags (vs clang passing them), this might be one of the few they do pass themselves, so having that unified across linkers seems valuable to me.

> BTW: for ELF, I'd like users specify `--fatal-warnings` instead of `-fatal-warnings`.

I think either option would be fine in this case if that's the recommendation


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113068



More information about the llvm-commits mailing list