[llvm-commits] CVS: llvm/include/Support/DenseMap.h

Alkis Evlogimenos alkis at uiuc.edu
Wed Feb 25 23:22:01 PST 2004


On Wednesday 25 February 2004 11:00 pm, Chris Lattner wrote:
> 
> Changes in directory llvm/include/Support:
> 
> DenseMap.h updated: 1.2 -> 1.3
> 
> ---
> Log message:
> 
> Fix a bug in the densemap that was killing the local allocator, and probably
> other clients.  The problem is that the nullVal member was left to the default
> constructor to initialize, which for int's does nothing (ie, leaves it unspecified).
> 
> To get a zero value, we must use T().  It's C++ wonderful? :)

Wow... I always thought that all members had T() called if you didn't specify it.

-- 

Alkis



More information about the llvm-commits mailing list