probinson added a comment. 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. https://reviews.llvm.org/D39896 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits