Endilll wrote: > > > We already have hard-coded logic to disable adding these tests to the > > > `check-all` target in various cases. > > > > > > Why do you have to do that? > > I've never worked with the clang python bindings before, but you can > basically see the reasoning in > https://github.com/llvm/llvm-project/blob/bf7022f71a95313e8ebdd34e5ea7cb7bca90a24d/clang/bindings/python/tests/CMakeLists.txt > - there are various targets and configurations where they are expected or > known to fail.
Sorry, I'm asking for the use case for the particular CMake knob you're adding. If there are more platforms that those tests are not working on, we should simply expand the pre-existing list https://github.com/llvm/llvm-project/blob/e2dc50c92987e6886b1b1641ebdbe9cc53bae192/clang/bindings/python/tests/CMakeLists.txt#L46 Or, better, disable tests individually based on the triple, like we do elsewhere. https://github.com/llvm/llvm-project/pull/111367 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
