This is an archive of the discontinued LLVM Phabricator instance.

Remove Windows-specific minidump plugin
ClosedPublic

Authored by amccarth on Nov 15 2016, 1:38 PM.

Details

Summary

With the cross-platform minidump plugin working, the Windows-specific one is no longer needed. This eliminates the unnecessary code.

This does not eliminate the Windows-specific tests, as they hit a few cases the general tests don't. (The Windows-specific tests are currently passing.) I'll look into a separate patch to make sure we're not doing too much duplicate testing.

After that I might do a little re-org in the Windows plugin, as there was some factoring there (Common & Live) that probably isn't necessary anymore.

Diff Detail

Repository
rL LLVM

Event Timeline

amccarth updated this revision to Diff 78065.Nov 15 2016, 1:38 PM
amccarth retitled this revision from to Remove Windows-specific minidump plugin.
amccarth updated this object.
amccarth added reviewers: labath, zturner.
amccarth added a subscriber: lldb-commits.
labath accepted this revision.Nov 16 2016, 2:25 AM
labath edited edge metadata.

Cool.

This revision is now accepted and ready to land.Nov 16 2016, 2:25 AM
This revision was automatically updated to reflect the committed changes.
lldb/trunk/source/Plugins/Process/Windows/MiniDump/x64/RegisterContextWindowsMiniDump_x64.h