[PATCH] D105330: [llvm-nm] Switch command line parsing from llvm::cl to OptTable
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 9 15:25:04 PDT 2021
MaskRay marked an inline comment as done.
MaskRay added inline comments.
================
Comment at: llvm/test/tools/llvm-nm/just-symbols.test:10
# RUN: llvm-nm --format=sysv -j %t.o | diff %t.txt -
-# RUN: llvm-nm -j --format=posix %t.o | diff %t.txt -
+# RUN: llvm-nm -j --format=posix %t.o | not diff -q %t.txt %t1.txt
----------------
hubert.reinterpretcast wrote:
> This is causing failures on our AIX builds:
> ```
> diff: illegal option -- q
> Usage: diff [-bitw] [[-C Lines|-D String|-e|-f|-n|-U Lines|-c|-u]|[-h]] File1 File2
> diff [-bilrstw] [[-C Lines|-e|-f|-n|-U Lines|-c|-u]|[-h]] [-S File] Directory1 Directory2
>
> ```
I removed -q.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105330/new/
https://reviews.llvm.org/D105330
More information about the llvm-commits
mailing list