diff options
author | Chris Lattner <sabre@nondot.org> | 2004-07-21 03:56:54 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2004-07-21 03:56:54 +0000 |
commit | bb8f43c8fcac4688f25c6d3a46651a0b1893eb03 (patch) | |
tree | f08c65bbc5832348fd63a33a79cab57e7c1fc7da /lib/Analysis/BasicAliasAnalysis.cpp | |
parent | adfe28d136ba403d3ccc65908a4db0385f62b72b (diff) |
Do not ignore casts unless they are pointer-pointer casts. This caused us
to miscompile the SingleSource/Regression/C++/pointer_member.cpp program.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@15062 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Analysis/BasicAliasAnalysis.cpp')
-rw-r--r-- | lib/Analysis/BasicAliasAnalysis.cpp | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/lib/Analysis/BasicAliasAnalysis.cpp b/lib/Analysis/BasicAliasAnalysis.cpp index f95adb3292..c0949079cc 100644 --- a/lib/Analysis/BasicAliasAnalysis.cpp +++ b/lib/Analysis/BasicAliasAnalysis.cpp @@ -240,10 +240,12 @@ BasicAliasAnalysis::alias(const Value *V1, unsigned V1Size, const Value *V2, unsigned V2Size) { // Strip off any constant expression casts if they exist if (const ConstantExpr *CE = dyn_cast<ConstantExpr>(V1)) - if (CE->getOpcode() == Instruction::Cast) + if (CE->getOpcode() == Instruction::Cast && + isa<PointerType>(CE->getOperand(0)->getType())) V1 = CE->getOperand(0); if (const ConstantExpr *CE = dyn_cast<ConstantExpr>(V2)) - if (CE->getOpcode() == Instruction::Cast) + if (CE->getOpcode() == Instruction::Cast && + isa<PointerType>(CE->getOperand(0)->getType())) V2 = CE->getOperand(0); // Are we checking for alias of the same value? @@ -255,9 +257,11 @@ BasicAliasAnalysis::alias(const Value *V1, unsigned V1Size, // Strip off cast instructions... if (const Instruction *I = dyn_cast<CastInst>(V1)) - return alias(I->getOperand(0), V1Size, V2, V2Size); + if (isa<PointerType>(I->getOperand(0)->getType())) + return alias(I->getOperand(0), V1Size, V2, V2Size); if (const Instruction *I = dyn_cast<CastInst>(V2)) - return alias(V1, V1Size, I->getOperand(0), V2Size); + if (isa<PointerType>(I->getOperand(0)->getType())) + return alias(V1, V1Size, I->getOperand(0), V2Size); // Figure out what objects these things are pointing to if we can... const Value *O1 = getUnderlyingObject(V1); |