[llvm-commits] [llvm] r46197 - /llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp
Duncan Sands
baldrick at free.fr
Sun Jan 20 02:49:28 PST 2008
Author: baldrick
Date: Sun Jan 20 04:49:23 2008
New Revision: 46197
URL: http://llvm.org/viewvc/llvm-project?rev=46197&view=rev
Log:
Initializing an unsigned with ~0UL causes the compiler
to complain on x86-64 (gcc 4.1). Use ~0U instead.
Modified:
llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp
Modified: llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp?rev=46197&r1=46196&r2=46197&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp Sun Jan 20 04:49:23 2008
@@ -211,11 +211,11 @@
Value* depPointer = dependency->getPointerOperand();
const Type* depType = dependency->getOperand(0)->getType();
unsigned depPointerSize = TD.getTypeStoreSize(depType);
-
+
// Check for aliasing
- AliasAnalysis::AliasResult A = AA.alias(F->getPointerOperand(), ~0UL,
+ AliasAnalysis::AliasResult A = AA.alias(F->getPointerOperand(), ~0U,
depPointer, depPointerSize);
-
+
if (A == AliasAnalysis::MustAlias) {
// Remove it!
MD.removeInstruction(dependency);
@@ -324,13 +324,13 @@
deadPointers.clear();
return MadeChange;
}
-
+
// Get size information for the alloca
- unsigned pointerSize = ~0UL;
+ unsigned pointerSize = ~0U;
if (ConstantInt* C = dyn_cast<ConstantInt>((*I)->getArraySize()))
pointerSize = C->getZExtValue() * \
TD.getABITypeSize((*I)->getAllocatedType());
-
+
// See if the call site touches it
AliasAnalysis::ModRefResult A = AA.getModRefInfo(CS, *I, pointerSize);
@@ -392,14 +392,14 @@
for (SmallPtrSet<AllocaInst*, 64>::iterator I = deadPointers.begin(),
E = deadPointers.end(); I != E; ++I) {
// Get size information for the alloca
- unsigned pointerSize = ~0UL;
+ unsigned pointerSize = ~0U;
if (ConstantInt* C = dyn_cast<ConstantInt>((*I)->getArraySize()))
pointerSize = C->getZExtValue() * \
TD.getABITypeSize((*I)->getAllocatedType());
-
+
// See if this pointer could alias it
AliasAnalysis::AliasResult A = AA.alias(*I, pointerSize,
- killPointer, ~0UL);
+ killPointer, ~0U);
// If it must-alias and a store, we can delete it
if (isa<StoreInst>(BBI) && A == AliasAnalysis::MustAlias) {
More information about the llvm-commits
mailing list