[PATCH] D94853: [NFC] Disallow unused prefixes under Other

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 19 11:35:04 PST 2021


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


================
Comment at: llvm/test/Other/lit.local.cfg:6-7
+# the default substitution of FileCheck will replace it to its full path.
+config.substitutions.insert(0, (fc.regex,
+    'FileCheck --allow-unused-prefixes=false'))
+config.substitutions.append(('%FileCheckRaw%', 'FileCheck'))
----------------
aeubanks wrote:
> pardon my ignorance, but why is this substitution necessary? does opt-bisect-legacy-pass-manager.ll break with `--allow-unused-prefixes=true`?
It's not that it breaks because of duplicate flags, it breaks because opt-bisect-helper.py has trouble starting a process called "FileCheck --allow-unused-prefixes=false"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94853



More information about the llvm-commits mailing list