[llvm-commits] [llvm] r53177 - in /llvm/trunk: include/llvm/ADT/DenseMap.h include/llvm/ADT/DenseSet.h lib/CodeGen/SelectionDAG/LegalizeDAG.cpp lib/CodeGen/SelectionDAG/ScheduleDAG.cpp lib/Linker/LinkModules.cpp lib/Target/X86/X86InstrInfo.cpp lib/Transforms/Scalar/InstructionCombining.cpp

Chris Lattner clattner at apple.com
Mon Jul 7 13:13:31 PDT 2008


On Jul 7, 2008, at 10:46 AM, Dan Gohman wrote:

> Author: djg
> Date: Mon Jul  7 12:46:23 2008
> New Revision: 53177
>
> URL: http://llvm.org/viewvc/llvm-project?rev=53177&view=rev
> Log:
> Make DenseMap's insert return a pair, to more closely resemble  
> std::map.

This is fine, but be careful: it is far easier to invalidate a  
densemap iterator than an std::map iterator.

-Chris




More information about the llvm-commits mailing list