Hey Todd, Did you run the tests earlier? If so what was the result? No worries if you didn't run them yet, but it sounded like you were already kicking it off at 11:30. Just want to make sure you didn't finish and forget to update with the result :)
On Wed, Nov 16, 2016 at 12:38 PM Todd Fiala <todd.fi...@gmail.com> wrote: > Yep - I followed that. I'm just doing the "build + test" verification on > macOS. > > > On Wed, Nov 16, 2016 at 11:41 AM, Zachary Turner <ztur...@google.com> > wrote: > > BTW, I would still like to get Chris to take a look at my usage of > llvm::Twine. Even if it works, I'm not sure if I used it correctly. > > On Wed, Nov 16, 2016 at 11:36 AM Zachary Turner <ztur...@google.com> > wrote: > > Either way is fine, I think you might have hit a merge conflict if you > stacked them, but if you've already worked through it, then no big deal. > > On Wed, Nov 16, 2016 at 11:30 AM Todd Fiala <todd.fi...@gmail.com> wrote: > > tfiala added a comment. > > I'm going to test this one now, stacked on top of the final macOS-working > version of https://reviews.llvm.org/D26698. Tell me now if you want it > tested independently of https://reviews.llvm.org/D26698. > > > https://reviews.llvm.org/D26721 > > > > > > > -- > -Todd >
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
