Re: [Lldb-commits] [PATCH] D39387: Invert ArchSpec<->Platform dependency

2017-10-27 Thread Zachary Turner via lldb-commits
On Fri, Oct 27, 2017 at 2:13 PM Jason Molenda wrote: > haven't had a chance to look at the patch yet, but just to comment on one > part from Zach: > > > On Oct 27, 2017, at 2:06 PM, Zachary Turner via Phabricator via > lldb-commits wrote: > > > > > > > > Comment at: source/Targe

Re: [Lldb-commits] [PATCH] D39387: Invert ArchSpec<->Platform dependency

2017-10-27 Thread Jason Molenda via lldb-commits
haven't had a chance to look at the patch yet, but just to comment on one part from Zach: > On Oct 27, 2017, at 2:06 PM, Zachary Turner via Phabricator via lldb-commits > wrote: > > > > Comment at: source/Target/Platform.cpp:986-991 > +if (normalized_triple.getVendorName(