[Lldb-commits] [PATCH] D57780: Don't include UnixSignals.h from Host
Zachary Turner via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Feb 5 12:52:39 PST 2019
zturner created this revision.
zturner added a reviewer: labath.
Herald added a reviewer: jfb.
This file lives in Target, which makes sense since it is supposed to represent the various types of signals that can occur on any platform (since you might be remote debugging a platform with different signals than the host).
However, Host had a method called `GetUnixSignals` which just returned an instance of this class initialized for the Host's signals. Rather than have this functionality in Host, we can just provide a static factory method on `UnixSignals` called `CreateForHost`. This brings us one step closer to breaking the Target -> Host -> Target cycle.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 4313 bytes
Desc: not available
More information about the lldb-commits