From fffdf105669c629333ce928bbf055cdf0815d1c8 Mon Sep 17 00:00:00 2001 From: Reid Spencer Date: Thu, 17 May 2007 06:26:29 +0000 Subject: For lshr by 0 bits, just return *this as a short cut. This also prevents undefined behavior when the width > 64 bits. llvm-svn: 37153 --- llvm/lib/Support/APInt.cpp | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'llvm/lib/Support') diff --git a/llvm/lib/Support/APInt.cpp b/llvm/lib/Support/APInt.cpp index 460cf557610..2ed9b07f1ea 100644 --- a/llvm/lib/Support/APInt.cpp +++ b/llvm/lib/Support/APInt.cpp @@ -1149,6 +1149,12 @@ APInt APInt::lshr(uint32_t shiftAmt) const { if (shiftAmt == BitWidth) return APInt(BitWidth, 0); + // If none of the bits are shifted out, the result is *this. This avoids + // issues with shifting byt he size of the integer type, which produces + // undefined results in the code below. This is also an optimization. + if (shiftAmt == 0) + return *this; + // Create some space for the result. uint64_t * val = new uint64_t[getNumWords()]; -- cgit v1.2.3