On Thu, Oct 18, 2012 at 1:45 PM, Douglas Gregor <dgre...@apple.com> wrote: > > On Oct 18, 2012, at 9:57 AM, David Blaikie <dblai...@gmail.com> wrote: > >> Author: dblaikie >> Date: Thu Oct 18 11:57:32 2012 >> New Revision: 166188 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=166188&view=rev >> Log: >> PR14021: Copy lookup results to ensure safe iteration. >> >> Within the body of the loop the underlying map may be modified via >> >> 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 ensure that this is the > only ticking time bomb in this area?
[+Robert Muth] I've not performed any such audit, no. With the hack debug check inserted we could just run the whole test suite to see if anything pops up. _______________________________________________ cfe-commits mailing list cfe-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits