6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-10-25 10:27:04 +00:00
Code Issues Projects Releases Wiki Activity

Fix a problem where we had bitcasted operands that gave us

Browse Source 
odd offsets since the bitcasted pointer size and the offset pointer
size are going to be different types for the GEP vs base object.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@96134 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Eric Christopher
2010-02-13 23:38:01 +00:00
parent 18c67b994c
commit 77ffe3b31e
2 changed files with 18 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
lib/Transforms/InstCombine/InstCombineCalls.cpp
View File

@@ -334,17 +334,21 @@ Instruction *InstCombiner::visitCallInst(CallInst &CI) {
// Make sure we're not a constant offset from an external
// global.
Value *Operand = GEP->getPointerOperand();
Operand = Operand->stripPointerCasts();
if (GlobalVariable *GV = dyn_cast<GlobalVariable>(Operand))
if (!GV->hasDefinitiveInitializer()) break;
// Get what we're pointing to and its size.
const PointerType *PT =
// Get what we're pointing to and its size.
const PointerType *BaseType =
cast<PointerType>(Operand->getType());
size_t Size = TD->getTypeAllocSize(PT->getElementType());
size_t Size = TD->getTypeAllocSize(BaseType->getElementType());
// Get the current byte offset into the thing.
// Get the current byte offset into the thing. Use the original
// operand in case we're looking through a bitcast.
SmallVector<Value*, 8> Ops(CE->op_begin()+1, CE->op_end());
size_t Offset = TD->getIndexedOffset(PT, &Ops[0], Ops.size());
const PointerType *OffsetType =
cast<PointerType>(GEP->getPointerOperand()->getType());
size_t Offset = TD->getIndexedOffset(OffsetType, &Ops[0], Ops.size());
assert(Size >= Offset);