[PATCH] D83578: [test] Replace a fragile lit feature (substitution in an argument place) with command -v

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

No particular opinion on this change itself, and I might be misunderstanding some of hte comments, but I wouldn't want to see the feature of expanding tool names to full paths to be dropped, at least for some cases. It is useful to me when I'm debugging a test as I can copy the tool path straight from the lit output to the command I wish to execute to see what actually is happening/tweak stuff/make sure I'm running what I think I'm running etc.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83578