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

[Jim Ingham via Phabricator via lldb-commits]

jingham added a comment.

Yes, my intention was not to gate this patch on figuring out the right role for Utility/Core/Whatever.  It just seemed an apropos moment to bring this up.

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.


https://reviews.llvm.org/D39896