[PATCH] D83069: [lit] warn if explicitly specified test won't be run indirectly

ben via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 3 02:07:43 PDT 2020


bd1976llvm added a comment.

In D83069#2129923 <https://reviews.llvm.org/D83069#2129923>, @jhenderson wrote:

> You said to me offline when you were discussing this that you were concerned by performance implications. I take it this isn't the approach you had then, or you changed your mind?


That was an earlier version (where I was creating a set of all possible tests and then checking that the specified test wasn't in it). Performance seems acceptable now.


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

https://reviews.llvm.org/D83069





More information about the llvm-commits mailing list