[PATCH] D65707: [FileCheck] Document FILECHECK_OPTS in -help

[Joel E. Denny via Phabricator via llvm-commits]

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


================
Comment at: llvm/utils/FileCheck/FileCheck.cpp:27-29
+static cl::extrahelp FileCheckOptsEnv(
+    "\nOptions are parsed from the environment variable FILECHECK_OPTS and\n"
+    "from the command line.\n");
----------------
thopre wrote:
> Do I read this right and the options in effect are the union of those specified in FILECHECK_OPTS and those on the command-line?
Yes.

Interestingly, `FILECHECK_OPTS` is parsed before command-line options.  However, at the time I implemented `FILECHECK_OPTS`, the only option I found where the order of multiple occurrences mattered was `-D`, and it gives precedence to earlier occurrences.  That's surely wrong, and I think multiple definitions for the same variable should be an error instead, but I haven't found time to propose something.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65707