From 5b7e659e220ed01c8d3a7335e2b52ff2f0fb2ef3 Mon Sep 17 00:00:00 2001 From: Reid Spencer Date: Sat, 24 Mar 2007 23:42:47 +0000 Subject: [PATCH] In the getBitsSet function, don't optimize for a common case that is already covered by getLowBitsSet (i.e. when loBits==0). Consequently, remove the default value for loBits and reorder the arguments to the more natural loBits, hiBits order. This makes it more clear that this function is for bit groups in the middle of the bit width and not towards one end or the other. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@35312 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/ADT/APInt.h | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/include/llvm/ADT/APInt.h b/include/llvm/ADT/APInt.h index fb6246d0084..b524b56e9b4 100644 --- a/include/llvm/ADT/APInt.h +++ b/include/llvm/ADT/APInt.h @@ -336,18 +336,16 @@ public: /// less than loBit then the set bits "wrap". For example, with /// parameters (32, 3, 28), you would get 0xF000000F. /// @param numBits the intended bit width of the result - /// @param hiBit the index of the highest bit set. /// @param loBit the index of the lowest bit set. + /// @param hiBit the index of the highest bit set. /// @returns An APInt value with the requested bits set. /// @brief Get a value with a block of bits set. - static APInt getBitsSet(uint32_t numBits, uint32_t hiBit, uint32_t loBit = 0){ + static APInt getBitsSet(uint32_t numBits, uint32_t loBit, uint32_t hiBit) { assert(hiBit < numBits && "hiBit out of range"); assert(loBit < numBits && "loBit out of range"); if (hiBit < loBit) return getLowBitsSet(numBits, hiBit+1) | getHighBitsSet(numBits, numBits-loBit+1); - else if (loBit == 0) - return getLowBitsSet(numBits, hiBit+1); return getLowBitsSet(numBits, hiBit-loBit+1).shl(loBit); } -- 2.34.1