[Lldb-commits] [PATCH] D58842: Move ProcessInstanceInfo and similar to Utility
Zachary Turner via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Mar 1 11:34:52 PST 2019
zturner created this revision.
zturner added reviewers: jingham, labath, JDevlieghere.
Herald added subscribers: jdoerfert, mgorny, emaste.
There are set of classes in Target that describe the parameters of a process - e.g. it's PID, name, user id, and similar. However, since it is a bare description of a process and contains no actual functionality, there's nothing specifically that makes this appropriate for being in Target -- it could just as well be describing a process on the host, or some hypothetical virtual process that doesn't even exist.
To cement this, I'm moving this class to Utility. It's possible that we can find a better place for it in the future, but as it is neither Host specific nor Target specific, Utility seems like the most appropriate place for the time being.
After this there are only 2 remaining references to Target from Host, neither of which is terribly difficult to fix on its own and which I'll address in followups.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 46970 bytes
Desc: not available
More information about the lldb-commits