This is an archive of the discontinued LLVM Phabricator instance.

Move StructuredData from Core to Utility
ClosedPublic

Authored by labath on Jun 26 2017, 8:23 AM.

Details

Summary

It had a dependency on StringConvert and file reading code, which is not
in Utility. I've replaced that code by equivalent llvm operations.

I've added a unit test to demonstrate that parsing a file still works.

Diff Detail

Repository
rL LLVM

Event Timeline

labath created this revision.Jun 26 2017, 8:23 AM
labath updated this revision to Diff 103976.Jun 26 2017, 9:40 AM

Well, PluginProcessPOSIX does not seem to be using anything from Core directly,
although I doubt that is thanks to this patch. It will be a long time before
anything stops depending on Core (my goal here is to do it for Host, but even
that is a long way off).

zturner accepted this revision.Jun 26 2017, 10:05 AM
This revision is now accepted and ready to land.Jun 26 2017, 10:05 AM
This revision was automatically updated to reflect the committed changes.
lldb/trunk/source/Utility/StructuredData.cpp