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 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
