[Lldb-commits] [PATCH] Use SmallPtrSet in ClusterManager
tfiala at google.com
Fri May 16 13:09:52 PDT 2014
I can go ahead and do this. I'm already at the point where I could check
it against Linux in a few minutes, might as well do that. If no errors,
I'll check it right in.
On Fri, May 16, 2014 at 12:48 PM, Scott Knight <knightsc at gmail.com> wrote:
> I do not have commit access so if someone who does could commit that would
> be great.
> Scott Knight
> On May 16, 2014 3:26 PM, <jingham at apple.com> wrote:
>> I got the same result. This patch is much more likely to cause crashes
>> than failures if it was wrong, and I didn't see that either. And the patch
>> looks reasonable. I say check it in.
>> On May 16, 2014, at 12:23 PM, Scott Knight <knightsc at gmail.com> wrote:
>> > I did go ahead and run dotest.py this morning both before and after
>> applying the patch and it seems to have the same 10 failures before and
>> after like you mentioned. Another set of eyes checking the test results
>> would be great though.
>> > http://reviews.llvm.org/D3616
>> > _______________________________________________
>> > lldb-commits mailing list
>> > lldb-commits at cs.uiuc.edu
>> > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
Todd Fiala | Software Engineer | tfiala at google.com | 650-943-3180
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the lldb-commits