Changes in directory llvm/lib/Transforms/IPO:
GlobalOpt.cpp updated: 1.98 -> 1.99 SimplifyLibCalls.cpp updated: 1.93 -> 1.94 --- Log message: Prefer non-virtual calls to ConstantInt::isZero over virtual calls to Constant::isNullValue() in situations where it is possible. --- Diffs of the changes: (+4 -4) GlobalOpt.cpp | 2 +- SimplifyLibCalls.cpp | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) Index: llvm/lib/Transforms/IPO/GlobalOpt.cpp diff -u llvm/lib/Transforms/IPO/GlobalOpt.cpp:1.98 llvm/lib/Transforms/IPO/GlobalOpt.cpp:1.99 --- llvm/lib/Transforms/IPO/GlobalOpt.cpp:1.98 Wed Feb 14 20:26:10 2007 +++ llvm/lib/Transforms/IPO/GlobalOpt.cpp Thu Mar 1 18:28:51 2007 @@ -1155,7 +1155,7 @@ // instead of a select to synthesize the desired value. bool IsOneZero = false; if (ConstantInt *CI = dyn_cast<ConstantInt>(OtherVal)) - IsOneZero = InitVal->isNullValue() && CI->equalsInt(1); + IsOneZero = InitVal->isNullValue() && CI->isOne(); while (!GV->use_empty()) { Instruction *UI = cast<Instruction>(GV->use_back()); Index: llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp diff -u llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp:1.93 llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp:1.94 --- llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp:1.93 Mon Feb 12 23:58:53 2007 +++ llvm/lib/Transforms/IPO/SimplifyLibCalls.cpp Thu Mar 1 18:28:51 2007 @@ -584,7 +584,7 @@ for (uint64_t i = 0; i < len; ++i) { if (ConstantInt* CI = dyn_cast<ConstantInt>(CA->getOperand(i))) { // Check for the null terminator - if (CI->isNullValue()) + if (CI->isZero()) break; // we found end of string else if (CI->getSExtValue() == chr) { char_found = true; @@ -2023,7 +2023,7 @@ // Check to make sure that the first operand of the GEP is an integer and // has value 0 so that we are sure we're indexing into the initializer. if (ConstantInt* op1 = dyn_cast<ConstantInt>(GEP->getOperand(1))) { - if (!op1->isNullValue()) + if (!op1->isZero()) return false; } else return false; @@ -2069,7 +2069,7 @@ for (len = start_idx; len < max_elems; len++) { if (ConstantInt *CI = dyn_cast<ConstantInt>(A->getOperand(len))) { // Check for the null terminator - if (CI->isNullValue()) + if (CI->isZero()) break; // we found end of string } else return false; // This array isn't suitable, non-int initializer _______________________________________________ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits