This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Make EXCLUDE_FROM_ALL an argument to add_lit_testsuite
ClosedPublic

Authored by JDevlieghere on Feb 6 2020, 2:10 PM.

Details

Summary

Rather than enabling the variable EXCLUDE_FROM_ALL, make this an argument to add_lit_testsuite.

Diff Detail

Event Timeline

JDevlieghere created this revision.Feb 6 2020, 2:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2020, 2:10 PM
Herald added a subscriber: mgorny. · View Herald Transcript

Make EXCLUDE_FROM_ALL an argument to add_lit_testsuite

Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2020, 2:27 PM
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.

Simplify logic in asan/CMakeLists.txt

JDevlieghere retitled this revision from [lldb\CMake] Exclude the reproducer test suites from check-all to [CMake] Make EXCLUDE_FROM_ALL an argument to add_lit_testsuite .Feb 6 2020, 2:59 PM
JDevlieghere edited the summary of this revision. (Show Details)

Rename the argument from EXCLUDE_FROM_ALL to EXCLUDE_FROM_CHECK_ALL as per Stella's suggestion.

stella.stamenova accepted this revision.Feb 6 2020, 3:23 PM

Thanks for doing this!

This revision is now accepted and ready to land.Feb 6 2020, 3:23 PM
This revision was automatically updated to reflect the committed changes.
Herald added projects: Restricted Project, Restricted Project, Restricted Project. · View Herald TranscriptFeb 6 2020, 3:40 PM
Herald added subscribers: openmp-commits, Restricted Project, cfe-commits. · View Herald Transcript
kschwarz added inline comments.
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?

JDevlieghere marked an inline comment as done.Feb 7 2020, 3:11 PM
JDevlieghere added inline comments.
llvm/test/CMakeLists.txt
171

Nope, I unintentionally inverted the condition. Thanks for pointing this out!

JDevlieghere marked an inline comment as done.Feb 7 2020, 3:18 PM
JDevlieghere added inline comments.
llvm/test/CMakeLists.txt
171

Fixed in c10b9f0bde2666abd3e2d0845dee16ac9db1ab6f