From 7c78afefe96669cc3eb886308291213415d9d7b0 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Thu, 26 Feb 2004 05:00:15 +0000 Subject: [PATCH] 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? :) git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11867 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/Support/DenseMap.h | 2 +- include/llvm/ADT/DenseMap.h | 2 +- include/llvm/ADT/IndexedMap.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/include/Support/DenseMap.h b/include/Support/DenseMap.h index 9d713d97c36..9d55435c91f 100644 --- a/include/Support/DenseMap.h +++ b/include/Support/DenseMap.h @@ -33,7 +33,7 @@ class DenseMap { ToIndexT toIndex_; public: - DenseMap() { } + DenseMap() : nullVal_(T()) { } explicit DenseMap(const T& val) : nullVal_(val) { } diff --git a/include/llvm/ADT/DenseMap.h b/include/llvm/ADT/DenseMap.h index 9d713d97c36..9d55435c91f 100644 --- a/include/llvm/ADT/DenseMap.h +++ b/include/llvm/ADT/DenseMap.h @@ -33,7 +33,7 @@ class DenseMap { ToIndexT toIndex_; public: - DenseMap() { } + DenseMap() : nullVal_(T()) { } explicit DenseMap(const T& val) : nullVal_(val) { } diff --git a/include/llvm/ADT/IndexedMap.h b/include/llvm/ADT/IndexedMap.h index 9d713d97c36..9d55435c91f 100644 --- a/include/llvm/ADT/IndexedMap.h +++ b/include/llvm/ADT/IndexedMap.h @@ -33,7 +33,7 @@ class DenseMap { ToIndexT toIndex_; public: - DenseMap() { } + DenseMap() : nullVal_(T()) { } explicit DenseMap(const T& val) : nullVal_(val) { }