DavidSpickett wrote:

53 tests use `@skipIf(compiler=no_match("clang"))`.

Then there are ones with version checks because of bugs in earlier clang 
versions. Those get more complex.

So for now I think I could land this PR as is, then follow up with a 
`requiresClang` for the simple cases. Does that sound good to you?

Btw, don't forget to update 
https://lldb.llvm.org/resources/test.html#guidelines-for-api-tests (can't link 
to it directly but scroll down, there's a skipif example) when you are done 
with your work on this.

https://github.com/llvm/llvm-project/pull/214197
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to