[llvm-commits] [llvm] r120772 - /llvm/trunk/include/llvm/Constants.h

Chris Lattner sabre at nondot.org
Thu Dec 2 17:11:13 PST 2010


Author: lattner
Date: Thu Dec  2 19:11:13 2010
New Revision: 120772

URL: http://llvm.org/viewvc/llvm-project?rev=120772&view=rev
Log:
clarify comment

Modified:
    llvm/trunk/include/llvm/Constants.h

Modified: llvm/trunk/include/llvm/Constants.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Constants.h?rev=120772&r1=120771&r2=120772&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Constants.h (original)
+++ llvm/trunk/include/llvm/Constants.h Thu Dec  2 19:11:13 2010
@@ -266,8 +266,8 @@
   inline const APFloat& getValueAPF() const { return Val; }
 
   /// isNullValue - Return true if this is the value that would be returned by
-  /// getNullValue.  Don't depend on == for doubles to tell us it's zero, it
-  /// considers -0.0 to be null as well as 0.0.  :(
+  /// getNullValue.  For ConstantFP, this is +0.0, but not -0.0.  To handle the
+  /// two the same, use isZero().
   virtual bool isNullValue() const;
   
   /// isNegativeZeroValue - Return true if the value is what would be returned 





More information about the llvm-commits mailing list