It's fine, just checking :)

lgtm

On Thu, Mar 16, 2017 at 7:49 AM Pavel Labath via Phabricator <
revi...@reviews.llvm.org> wrote:

> labath added a comment.
>
> In https://reviews.llvm.org/D31031#702747, @zturner wrote:
>
> > You mention that this was the cause of dependencies from `Host` to
> `Target`, but I don't see any `#include "lldb/Target/*.h"` statements
> removed.  Is this an oversight or not possible yet?
>
>
> I guess I wasn't clear about that. One of the GetAuxvData overloads was
> taking a `Process*` argument, but this is by no means the last dependency
> on stuff from the Target module.
>
>
> https://reviews.llvm.org/D31031
>
>
>
>
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to