[Lldb-commits] [PATCH] D34625: Move StructuredData from Core to Utility

2017-06-27 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL306394: Move StructuredData from Core to Utility (authored by labath). Changed prior to commit: https://reviews.llvm.org/D34625?vs=103976&id=104127#toc Repository: rL LLVM https://reviews.llvm.org/D

[Lldb-commits] [PATCH] D34625: Move StructuredData from Core to Utility

2017-06-26 Thread Pavel Labath via Phabricator via lldb-commits
labath updated this revision to Diff 103976. labath added a comment. Herald added a subscriber: emaste. 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

Re: [Lldb-commits] [PATCH] D34625: Move StructuredData from Core to Utility

2017-06-26 Thread Zachary Turner via lldb-commits
Can you run the analyze deps script before and after this patch, and update the cmake files if any deps are no longer necessary? On Mon, Jun 26, 2017 at 8:24 AM Pavel Labath via Phabricator < revi...@reviews.llvm.org> wrote: > labath created this revision. > Herald added subscribers: mgorny, kubam

[Lldb-commits] [PATCH] D34625: Move StructuredData from Core to Utility

2017-06-26 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. Herald added subscribers: mgorny, kubamracek. 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. https://