6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2024-10-01 10:57:21 +00:00
Code Issues Projects Releases Wiki Activity

Don't commit stores with addresses that have indices that are not

Browse Source 
compile-time constant integers or that are out of bounds for their
corresponding static array types. These can cause aliasing that
GlobalOpt assumes won't happen.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@81165 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Dan Gohman 2009-09-07 22:44:55 +00:00
parent ce5de5b527
commit 80bdc967e9
1 changed files with 21 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
lib/Transforms/IPO/GlobalOpt.cpp
View File

@ -2044,6 +2044,27 @@ static bool isSimpleEnoughPointerToCommit(Constant *C, LLVMContext &Context) {
// external globals.
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);
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;
}
return ConstantFoldLoadThroughGEPConstantExpr(GV->getInitializer(), CE,
Context);
}