[PATCH] D95849: [FileCheck] Default --allow-unused-prefixes to false

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 4 11:32:11 PST 2021


MaskRay added subscribers: pengfei, RKSimon.
MaskRay added a comment.

@nikic D95849 <https://reviews.llvm.org/D95849> has already changed `llvm/test/` to default to `--allow-unused-prefixes`.  I updated a few directories to be `--allow-unused-prefixes` clean (this uncovered improvement to many tests).

Some `update{,_llc,_analyzer}_test_checks.py` use cases may find `--allow-unused-prefixes=false` mode better, but such use cases still appear to be a minority (e.g. @RKSimon/@pengfei/my/others' fixes to test/CodeGen directories).
https://lists.llvm.org/pipermail/llvm-dev/2021-February/148331.html


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95849



More information about the cfe-commits mailing list