The testing to ensure a vector of zeros of type floating point isn't misclassified...
authorDavid Tweed <david.tweed@arm.com>
Tue, 19 Mar 2013 10:16:40 +0000 (10:16 +0000)
committerDavid Tweed <david.tweed@arm.com>
Tue, 19 Mar 2013 10:16:40 +0000 (10:16 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@177386 91177308-0d34-0410-b5e6-96231b3b80d8

lib/IR/Constants.cpp

index 70f7e0176e85de464c6c71da6697e1cae0836355..1abb65643559ad966ce0e10e17494c77f71a37e0 100644 (file)
@@ -53,12 +53,9 @@ bool Constant::isNegativeZeroValue() const {
       if (SplatCFP && SplatCFP->isZero() && SplatCFP->isNegative())
         return true;
 
-  // However, vectors of zeroes which are floating point represent +0.0's.
-  if (const ConstantAggregateZero *CAZ = dyn_cast<ConstantAggregateZero>(this))
-    if (const VectorType *VT = dyn_cast<VectorType>(CAZ->getType()))
-      if (VT->getElementType()->isFloatingPointTy())
-        // As it's a CAZ, we know it's the zero bit-pattern (ie, +0.0) in each element.
-        return false;
+  // We've already handled true FP case; any other FP vectors can't represent -0.0.
+  if (getType()->isFPOrFPVectorTy())
+    return false;
 
   // Otherwise, just use +0.0.
   return isNullValue();