[PATCH] D96662: [lit] Add --skip (inverse of --filter) and `--xfail`

Joel E. Denny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 19 01:47:52 PST 2021


jdenny added a comment.

In D96662#2572321 <https://reviews.llvm.org/D96662#2572321>, @yln wrote:

> `--skip`: is it okay if we rename this to `--filter-not`? It's an uglier name but

I like this suggestion for all the reasons @yln gave. However, I'd prefer --filter-out. It has the same advantages @yln gave, such as being obvious if you know what --filter does.  A small disadvantage is lack of grammatical symmetry. My issue is that --filter is not a clear name whereas --filter-in would've been clear.   Without that context, --filter-not is worse, in my opinion.  It sounds like "don't filter these tests", but isn't that the meaning of --filter?  --filter-out is clear in any context, in my opinion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96662



More information about the llvm-commits mailing list