[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