[Lldb-commits] [PATCH] D39896: Remove last Host usage from ArchSpec

[Zachary Turner via lldb-commits]

Super awesome. When you do move it to Utility, can you run the deps python
script to see if any cmake dependencies can be updated?
On Fri, Nov 10, 2017 at 4:02 AM Pavel Labath via Phabricator <
reviews at reviews.llvm.org> wrote:

> labath created this revision.
>
> In https://reviews.llvm.org/D39387, I was quick to jump to conclusion
> that ArchSpec has no
> external dependencies. It turns there still was one call to
> HostInfo::GetArchitecture left -- for implementing the "systemArch32"
> architecture and friends.
>
> Since GetAugmentedArchSpec is the place we handle these "incomplete"
> triples that don't specify os or vendor and "systemArch" looks very much
> like an incomplete triple, I move its handling there.
>
> After this ArchSpec *really* does not have external dependencies, and
> I'd like to move it to the Utility module as a follow-up.
>
>
> https://reviews.llvm.org/D39896
>
> Files:
>   include/lldb/Host/HostInfoBase.h
>   source/Core/ArchSpec.cpp
>   source/Host/common/HostInfoBase.cpp
>   source/Target/Platform.cpp
>   unittests/Host/HostInfoTest.cpp
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20171110/450c41ba/attachment.html>