[PATCH] D119394: [lld-macho][nfc] Centralize usages of ld64.lld in tests
Vincent Lee via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 9 17:54:24 PST 2022
thevinster added a comment.
> why doesn't re-using %lld plus additional args, which would override the defalut ones provided by %lld anyways, work?
The substitution happens in order and it sees `%lld` within `%lld-noarg` first, so it ends up replacing that whole command and the command ends up failing a whole bunch of tests with `ld64.lld: error: unknown argument '-fatal_warnings-raw'`. So, moving that statement up should solve the issue. If the majority feels `%lld-noarg` is a better fit, I'm happy to change it to that; although, isn't noarg technically two words? (So, it should be `%lld-no-arg`?)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119394/new/
https://reviews.llvm.org/D119394
More information about the llvm-commits
mailing list