[Lldb-commits] [PATCH] D24629: Allow for tests to be disabled at runtime

[Francis Ricci via lldb-commits]

fjricci added a comment.

I do understand the complexity problem, and it was one of my concerns with this as well. For my cases, the complexity here is significantly less than the alternatives, but I also do understand if you don't think that's generally true.

It probably comes down to how often we think that people are running the test suite in cases where this sort of functionality would be useful. I don't really have a good sense for how other people tend to use the test suite, so I'm personally not sure.  For our case, it's a big deal, but if we're the only people who this patch helps, I know it doesn't make sense to merge it.


https://reviews.llvm.org/D24629