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

[Zachary Turner via Phabricator via lldb-commits]

zturner added a comment.

In https://reviews.llvm.org/D39896#922381, @probinson wrote:

> Drive by comment:
>
> In https://reviews.llvm.org/D39896#922224, @jingham wrote:
>
> > You're right, the Triple stuff is in ADT!  I was using it as an example of something you clearly wouldn't do so I didn't actually check to see if it was done.  That's pretty funny.
> >
> > I thought that ADT stood for Abstract Data Types - though I actually I don't remember where I got that impression...  Having Triple there does seem confusing.  That is a "how we specify Targets type thing", not a fancy data type type thing...  I would never think to look for that facility alongside StringRef & DenseMap etc.
>
>
> Triple.h is in ADT; Triple.cpp is in Support.  I agree it belongs in Support, and I'm not sure why it's schizo like that.


Ok now that's just crazy.  You're making me want to go fix this...


https://reviews.llvm.org/D39896