- new SROA mallocs should have the mallocs running-or'ed, not the malloc's bitcast

- fix ProcessInternalGlobal() debug output



git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@86317 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Victor Hernandez
2009-11-07 00:41:19 +00:00
parent 4d81d97a06
commit 631f3c0dfd

View File

@@ -1303,7 +1303,9 @@ static GlobalVariable *PerformHeapAllocSRoA(GlobalVariable *GV, CallInst *CI,
ConstantInt::get(IntPtrTy, TypeSize), ConstantInt::get(IntPtrTy, TypeSize),
NElems, NElems,
CI->getName() + ".f" + Twine(FieldNo)); CI->getName() + ".f" + Twine(FieldNo));
FieldMallocs.push_back(NMI); CallInst *NCI = dyn_cast<BitCastInst>(NMI) ?
extractMallocCallFromBitCast(NMI) : cast<CallInst>(NMI);
FieldMallocs.push_back(NCI);
new StoreInst(NMI, NGV, CI); new StoreInst(NMI, NGV, CI);
} }
@@ -1530,8 +1532,7 @@ static bool TryToOptimizeStoreOfMallocToGlobal(GlobalVariable *GV,
CI->replaceAllUsesWith(Cast); CI->replaceAllUsesWith(Cast);
CI->eraseFromParent(); CI->eraseFromParent();
CI = dyn_cast<BitCastInst>(Malloc) ? CI = dyn_cast<BitCastInst>(Malloc) ?
extractMallocCallFromBitCast(Malloc): extractMallocCallFromBitCast(Malloc) : cast<CallInst>(Malloc);
cast<CallInst>(Malloc);
} }
GVI = PerformHeapAllocSRoA(GV, CI, getMallocArraySize(CI, TD), TD); GVI = PerformHeapAllocSRoA(GV, CI, getMallocArraySize(CI, TD), TD);
@@ -1688,24 +1689,26 @@ bool GlobalOpt::ProcessInternalGlobal(GlobalVariable *GV,
if (!AnalyzeGlobal(GV, GS, PHIUsers)) { if (!AnalyzeGlobal(GV, GS, PHIUsers)) {
#if 0 #if 0
cerr << "Global: " << *GV; DEBUG(errs() << "Global: " << *GV);
cerr << " isLoaded = " << GS.isLoaded << "\n"; DEBUG(errs() << " isLoaded = " << GS.isLoaded << "\n");
cerr << " StoredType = "; DEBUG(errs() << " StoredType = ");
switch (GS.StoredType) { switch (GS.StoredType) {
case GlobalStatus::NotStored: cerr << "NEVER STORED\n"; break; case GlobalStatus::NotStored: DEBUG(errs() << "NEVER STORED\n"); break;
case GlobalStatus::isInitializerStored: cerr << "INIT STORED\n"; break; case GlobalStatus::isInitializerStored: DEBUG(errs() << "INIT STORED\n");
case GlobalStatus::isStoredOnce: cerr << "STORED ONCE\n"; break; break;
case GlobalStatus::isStored: cerr << "stored\n"; break; case GlobalStatus::isStoredOnce: DEBUG(errs() << "STORED ONCE\n"); break;
case GlobalStatus::isStored: DEBUG(errs() << "stored\n"); break;
} }
if (GS.StoredType == GlobalStatus::isStoredOnce && GS.StoredOnceValue) if (GS.StoredType == GlobalStatus::isStoredOnce && GS.StoredOnceValue)
cerr << " StoredOnceValue = " << *GS.StoredOnceValue << "\n"; DEBUG(errs() << " StoredOnceValue = " << *GS.StoredOnceValue << "\n");
if (GS.AccessingFunction && !GS.HasMultipleAccessingFunctions) if (GS.AccessingFunction && !GS.HasMultipleAccessingFunctions)
cerr << " AccessingFunction = " << GS.AccessingFunction->getName() DEBUG(errs() << " AccessingFunction = " << GS.AccessingFunction->getName()
<< "\n"; << "\n");
cerr << " HasMultipleAccessingFunctions = " DEBUG(errs() << " HasMultipleAccessingFunctions = "
<< GS.HasMultipleAccessingFunctions << "\n"; << GS.HasMultipleAccessingFunctions << "\n");
cerr << " HasNonInstructionUser = " << GS.HasNonInstructionUser<<"\n"; DEBUG(errs() << " HasNonInstructionUser = "
cerr << "\n"; << GS.HasNonInstructionUser<<"\n");
DEBUG(errs() << "\n");
#endif #endif
// If this is a first class global and has only one accessing function // If this is a first class global and has only one accessing function