This is an archive of the discontinued LLVM Phabricator instance.

Remove Go debugger plugin
ClosedPublic

Authored by JDevlieghere on Nov 2 2018, 4:21 PM.

Details

Summary

In January Davide sent an e-mail to the mailing list to suggest removing support for Go and Java. The plan was to have some cool down period to allow users to speak up. However, the actual removal never took place. After having the Go test fail on my machine since forever I decided to move forward and start the removal process.

Discussion on the mailing list: http://lists.llvm.org/pipermail/lldb-dev/2018-January/013171.html

Diff Detail

Event Timeline

JDevlieghere created this revision.Nov 2 2018, 4:21 PM

Missed some files.

tberghammer accepted this revision.Nov 5 2018, 1:49 AM
tberghammer added a reviewer: ribrdb.

LGTM. Adding Ryan as a reviewer as he was the original implementer but I am not sure if his account (and the linked e-mail) is still active.

This revision is now accepted and ready to land.Nov 5 2018, 1:49 AM
davide accepted this revision.Nov 5 2018, 8:11 AM

LGTM. Thanks for picking up the slack.

This revision was automatically updated to reflect the committed changes.
tools/lldb-test/SystemInitializerTest.cpp