[Lldb-commits] [PATCH] D58971: Move MemoryRegionInfo into the Utility module

2019-03-06 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. In D58971#1418843 , @zturner wrote: > One idea for a new module name could be `AbstractProcess`, but I can't think > of anything else better at the moment. Yeah, naming the new module might be the hardest problem here. :) I'd cal

[Lldb-commits] [PATCH] D58971: Move MemoryRegionInfo into the Utility module

2019-03-05 Thread Zachary Turner via Phabricator via lldb-commits
zturner added a comment. One idea for a new module name could be `AbstractProcess`, but I can't think of anything else better at the moment. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58971/new/ https://reviews.llvm.org/D58971 ___ lldb-

[Lldb-commits] [PATCH] D58971: Move MemoryRegionInfo into the Utility module

2019-03-05 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere added a comment. I like the idea of having a separate module for this. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58971/new/ https://reviews.llvm.org/D58971 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://l

[Lldb-commits] [PATCH] D58971: Move MemoryRegionInfo into the Utility module

2019-03-05 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. labath added reviewers: zturner, jingham, JDevlieghere, clayborg. Herald added a subscriber: mgorny. labath added a parent revision: D58970: Move RangeMap.h into Utility. MemoryRegionInfo describes the region of memory in a process, but this does not have to be the li