[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