This is an archive of the discontinued LLVM Phabricator instance.

[lldb] removed no longer needed CMakeDependentOption
ClosedPublic

Authored by kwk on Feb 6 2020, 1:22 PM.

Details

Summary

In D66791 I've introduced this [[ https://cmake.org/cmake/help/latest/module/CMakeDependentOption.html | CMakeDependentOption ]] but in
D71306 @JDevlieghere has changed the way optional dependencies
are handled in LLDB. Today there's no occurence of
cmake_dependent_option inside the lldb source tree.

That's why this include can be removed.

Diff Detail

Event Timeline

kwk created this revision.Feb 6 2020, 1:22 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2020, 1:22 PM
kwk updated this revision to Diff 242991.Feb 6 2020, 1:23 PM

Removed typo in commit message

kwk edited the summary of this revision. (Show Details)Feb 6 2020, 1:23 PM
kwk edited the summary of this revision. (Show Details)
Harbormaster completed remote builds in B45889: Diff 242991.
labath accepted this revision.Feb 6 2020, 1:34 PM
labath added a subscriber: labath.

ship it

This revision is now accepted and ready to land.Feb 6 2020, 1:34 PM
This revision was automatically updated to reflect the committed changes.