[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