[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