Re: [llvm-commits] [llvm] r41074 - /llvm/trunk/include/llvm/Analysis/Dominators.h

2007-08-14 Thread Nick Lewycky
On Tue, 2007-08-14 at 16:53 +, Devang Patel wrote: > +for (iterator I = begin(), E = end(); I != E; ++I) > + if (I->second.count(BB)) > +I->second.erase(BB); > +Frontiers.erase(BB); >} Why test I->second.count()? The worst erase() will do if BB doesn't exist is nothing

[llvm-commits] [llvm] r41074 - /llvm/trunk/include/llvm/Analysis/Dominators.h

2007-08-14 Thread Devang Patel
Author: dpatel Date: Tue Aug 14 11:53:24 2007 New Revision: 41074 URL: http://llvm.org/viewvc/llvm-project?rev=41074&view=rev Log: No need to use iterator to erase basic block. Modified: llvm/trunk/include/llvm/Analysis/Dominators.h Modified: llvm/trunk/include/llvm/Analysis/Dominators.h URL