davide accepted this revision.
davide added a comment.
This revision is now accepted and ready to land.
No objections from me.
https://reviews.llvm.org/D45480
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
labath added a comment.
In https://reviews.llvm.org/D45480#1063268, @zturner wrote:
> > This removes the last (direct) dependency from the Host module to
> > Interpreter, so I remove the Interpreter module from Host's dependency list.
>
> Big milestone! Kudos
Thanks. Host is still a member
zturner added a comment.
> This removes the last (direct) dependency from the Host module to
> Interpreter, so I remove the Interpreter module from Host's dependency list.
Big milestone! Kudos
https://reviews.llvm.org/D45480
___
lldb-commits
labath created this revision.
labath added reviewers: zturner, jingham, davide.
Herald added a subscriber: mgorny.
The Args class is used in plenty of places besides the command
interpreter (e.g., anything requiring an argc+argv combo, such as when
launching a process), so it needs to be in a
