[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