From 8bf92329eeea76e4e01c49f31e68c29cdada5985 Mon Sep 17 00:00:00 2001 From: Nilay Vaish Date: Tue, 14 Jun 2011 19:51:44 -0500 Subject: [PATCH] Ruby: Correct set LONG_BITS and INDEX_SHIFT in class Set. The code for Set class was written under the assumption that std::numeric_limits::digits returns the number of bits used for data type long, which was presumed to be either 32 or 64. But return value is actually one less, that is, it is either 31 or 63. The value is now being incremented by 1 so as to correctly set it. --- src/mem/ruby/common/Set.hh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mem/ruby/common/Set.hh b/src/mem/ruby/common/Set.hh index 5481c2ec7..b76c3409d 100644 --- a/src/mem/ruby/common/Set.hh +++ b/src/mem/ruby/common/Set.hh @@ -54,7 +54,7 @@ class Set long *m_p_nArray; // an word array to hold the bits in the set long m_p_nArray_Static[NUMBER_WORDS_PER_SET]; - static const int LONG_BITS = std::numeric_limits::digits; + static const int LONG_BITS = std::numeric_limits::digits + 1; static const int INDEX_SHIFT = LONG_BITS == 64 ? 6 : 5; static const int INDEX_MASK = (1 << INDEX_SHIFT) - 1; -- 2.30.2