Rather than enabling the variable EXCLUDE_FROM_ALL, make this an argument to add_lit_testsuite.
Details
Diff Detail
Event Timeline
lldb/test/Shell/CMakeLists.txt | ||
---|---|---|
24 | After the change you made to add_lit_testsuite, this can be anywhere in the function call - including the end, if you so desire. |
Update all uses of add_lit_testsuite to pass EXCLUDE_FROM_ALL as an argument rather than a scoped variable.
Rename the argument from EXCLUDE_FROM_ALL to EXCLUDE_FROM_CHECK_ALL as per Stella's suggestion.
llvm/test/CMakeLists.txt | ||
---|---|---|
171 | Hi @JDevlieghere, we've noticed that with this patch check-llvm isn't added to check-all anymore by default. Is this the intended behaviour? |
llvm/test/CMakeLists.txt | ||
---|---|---|
171 | Nope, I unintentionally inverted the condition. Thanks for pointing this out! |
llvm/test/CMakeLists.txt | ||
---|---|---|
171 | Fixed in c10b9f0bde2666abd3e2d0845dee16ac9db1ab6f |
After the change you made to add_lit_testsuite, this can be anywhere in the function call - including the end, if you so desire.