[PATCH] D82923: introducing llvm-libtool-darwin

Thu Jul 9 07:52:53 PDT 2020

sameerarora101 marked an inline comment as done.
sameerarora101 added inline comments.

================
Comment at: llvm/test/tools/llvm-libtool-darwin/help-message.test:5
+# RUN: llvm-libtool-darwin -help | FileCheck --check-prefixes=LIBTOOL-USAGE,CATEG %s --match-full-lines
+# RUN: llvm-libtool-darwin --help | FileCheck --check-prefixes=LIBTOOL-USAGE,CATEG %s --match-full-lines
+# RUN: llvm-libtool-darwin --help-list | \
----------------
jhenderson wrote:
> It's quite possible the llvm-size test doesn't have testing for unrelated options. How is it tested in this version now?
yup, you are right, `llvm-size` doesn't test that unrelated options are not present. For my case, the unrelated option `--safepoint-ir-verifier-print-only:` comes under `General Options:` when I add the support for `-static`. I can add `--implicit-check-not=General` in the second diff D83002 for that?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82923