Page MenuHomePhabricator

[lldb] removed no longer needed CMakeDependentOption
ClosedPublic

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

Details

Summary

In D66791 I've introduced this 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.Thu, Feb 6, 1:22 PM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Feb 6, 1:22 PM
kwk updated this revision to Diff 242991.Thu, Feb 6, 1:23 PM

Removed typo in commit message

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

ship it

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