[cfe-commits] clang patch for bug 14021

2012-10-15 Thread Robert Muth
This is a fix for: http://llvm.org/bugs/show_bug.cgi?id=14021 diff --git a/lib/Sema/SemaInit.cpp b/lib/Sema/SemaInit.cpp index 42fc84d..d4d0e94 100644 --- a/lib/Sema/SemaInit.cpp +++ b/lib/Sema/SemaInit.cpp @@ -3708,8 +3708,14 @@ static void TryUserDefinedConversion(Sema &S, // Try to c

Re: [cfe-commits] clang patch for bug 14021

2012-10-15 Thread Robert Muth
No small test, sadly. I managed to cut a 10MB reproducer down to 2MB over 2h but no further. On Mon, Oct 15, 2012 at 6:44 PM, Nico Weber wrote: > Test? > > On Mon, Oct 15, 2012 at 3:10 PM, Robert Muth wrote: >> This is a fix for: http://llvm.org/bugs/show_bug.cgi?id=14021 >>

Re: [cfe-commits] clang patch for bug 14021

2012-10-16 Thread Robert Muth
On Mon, Oct 15, 2012 at 10:57 PM, Rafael EspĂ­ndola wrote: > On 15 October 2012 19:02, Robert Muth wrote: >> No small test, sadly. >> I managed to cut a 10MB reproducer down to 2MB over 2h >> but no further. > > The testcase in the bug report is already a lot smaller t

Re: [cfe-commits] clang patch for bug 14021

2012-10-16 Thread Robert Muth
On Tue, Oct 16, 2012 at 11:47 AM, David Blaikie wrote: > On Tue, Oct 16, 2012 at 8:12 AM, Robert Muth wrote: >> On Mon, Oct 15, 2012 at 10:57 PM, Rafael EspĂ­ndola >> wrote: >>> On 15 October 2012 19:02, Robert Muth wrote: >>>> No small test, sadly. >>

Re: [cfe-commits] clang patch for bug 14021

2012-10-18 Thread Robert Muth
On Tue, Oct 16, 2012 at 1:47 PM, David Blaikie wrote: > On Tue, Oct 16, 2012 at 10:09 AM, Robert Muth wrote: >> On Tue, Oct 16, 2012 at 11:47 AM, David Blaikie wrote: >>> On Tue, Oct 16, 2012 at 8:12 AM, Robert Muth wrote: >>>> On Mon, Oct 15, 2012 at 10:57 PM

Re: [cfe-commits] r166188 - /cfe/trunk/lib/Sema/SemaInit.cpp

2012-10-18 Thread Robert Muth
> Sema::AddOverloadCandidate >>>-> Sema::CompareReferenceRelationship >>>-> Sema::RequireCompleteType >>> >>> to avoid the use of invalid iterators the sequence is copied first. >> >> Did you audit other uses of LookupConstructors to ens

Re: [cfe-commits] r166188 - /cfe/trunk/lib/Sema/SemaInit.cpp

2012-10-19 Thread Robert Muth
I started working on this - I'll post updates to this thread. On Fri, Oct 19, 2012 at 1:21 PM, David Blaikie wrote: > On Fri, Oct 19, 2012 at 10:10 AM, Matthieu Monrocq > wrote: >> >> >> On Thu, Oct 18, 2012 at 11:59 PM, Robert Muth wrote: >>> >>>

Re: [cfe-commits] r166188 - /cfe/trunk/lib/Sema/SemaInit.cpp

2012-10-29 Thread Robert Muth
invalidated. Unfortunately, they are not real iterators but just "pointers", so there is no easy way to hook them. I will keep looking into this but this will likely take more time. On Fri, Oct 19, 2012 at 1:26 PM, Robert Muth wrote: > I started working on this - I'll post up