[PATCH] D124445: [llvm-ar] Modify usage printouts to use the correct toolname
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 3 07:48:38 PDT 2022
jhenderson added inline comments.
================
Comment at: llvm/test/tools/llvm-ar/tool-name.test:13
+# RUN: %t/ar.exe h | FileCheck %s --check-prefix=SUFFIX
+## Ensure that the lib substring does not result in misidentification as the
+## llvm-lib tool.
----------------
I'd quote the "lib" word, as per the suggested edit: this should make it clear that "lib" is actually the substring, and not that there's some "lib substring".
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124445/new/
https://reviews.llvm.org/D124445
More information about the llvm-commits
mailing list