6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2026-04-26 12:20:42 +00:00
Code Issues Projects Releases Wiki Activity

Factor out the code for checking that all indices in a getelementptr are

Browse Source 
within the notional bounds of the static type of the getelementptr (which
is not the same as "inbounds") from GlobalOpt into a utility routine,
and use it in ConstantFold.cpp to check whether there are any mis-behaved
indices.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@81478 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Dan Gohman
2009-09-10 23:37:55 +00:00
parent 83e3c4f9d7
commit e6992f728a
5 changed files with 54 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/Transforms/IPO/GlobalOpt.cpp
+4 -15
View File
@@ -2045,25 +2045,14 @@ static bool isSimpleEnoughPointerToCommit(Constant *C, LLVMContext &Context) {
if (!GV->hasDefinitiveInitializer())
return false;
gep_type_iterator GEPI = gep_type_begin(CE), E = gep_type_end(CE);
User::op_iterator OI = next(CE->op_begin());
// The first index must be zero.
ConstantInt *CI = dyn_cast<ConstantInt>(*OI);
ConstantInt *CI = dyn_cast<ConstantInt>(*next(CE->op_begin()));
if (!CI || !CI->isZero()) return false;
++GEPI;
++OI;
// The remaining indices must be compile-time known integers within the
// bounds of the corresponding static array types.
for (; GEPI != E; ++GEPI, ++OI) {
CI = dyn_cast<ConstantInt>(*OI);
if (!CI) return false;
if (const ArrayType *ATy = dyn_cast<ArrayType>(*GEPI))
if (CI->getValue().getActiveBits() > 64 ||
CI->getZExtValue() >= ATy->getNumElements())
return false;
}
// notional bounds of the corresponding static array types.
if (!CE->isGEPWithNoNotionalOverIndexing())
return false;
return ConstantFoldLoadThroughGEPConstantExpr(GV->getInitializer(), CE,
Context);