[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