mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2025-07-23 14:25:07 +00:00
[PM/AA] Start refactoring AliasAnalysis to remove the analysis group and
port it to the new pass manager. All this does is extract the inner "location" class used by AA into its own full fledged type. This seems *much* cleaner as MemoryDependence and soon MemorySSA also use this heavily, and it doesn't make much sense being inside the AA infrastructure. This will also make it much easier to break apart the AA infrastructure into something that stands on its own rather than using the analysis group design. There are a few places where this makes APIs not make sense -- they were taking an AliasAnalysis pointer just to build locations. I'll try to clean those up in follow-up commits. Differential Revision: http://reviews.llvm.org/D10228 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@239003 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
@@ -241,7 +241,7 @@ bool MergedLoadStoreMotion::isDiamondHead(BasicBlock *BB) {
|
||||
bool MergedLoadStoreMotion::isLoadHoistBarrierInRange(const Instruction& Start,
|
||||
const Instruction& End,
|
||||
LoadInst* LI) {
|
||||
AliasAnalysis::Location Loc = AA->getLocation(LI);
|
||||
AliasAnalysis::Location Loc = MemoryLocation::get(LI);
|
||||
return AA->canInstructionRangeModRef(Start, End, Loc, AliasAnalysis::Mod);
|
||||
}
|
||||
|
||||
@@ -266,8 +266,8 @@ LoadInst *MergedLoadStoreMotion::canHoistFromBlock(BasicBlock *BB1,
|
||||
LoadInst *Load1 = dyn_cast<LoadInst>(Inst);
|
||||
BasicBlock *BB0 = Load0->getParent();
|
||||
|
||||
AliasAnalysis::Location Loc0 = AA->getLocation(Load0);
|
||||
AliasAnalysis::Location Loc1 = AA->getLocation(Load1);
|
||||
AliasAnalysis::Location Loc0 = MemoryLocation::get(Load0);
|
||||
AliasAnalysis::Location Loc1 = MemoryLocation::get(Load1);
|
||||
if (AA->isMustAlias(Loc0, Loc1) && Load0->isSameOperationAs(Load1) &&
|
||||
!isLoadHoistBarrierInRange(BB1->front(), *Load1, Load1) &&
|
||||
!isLoadHoistBarrierInRange(BB0->front(), *Load0, Load0)) {
|
||||
@@ -425,8 +425,8 @@ StoreInst *MergedLoadStoreMotion::canSinkFromBlock(BasicBlock *BB1,
|
||||
|
||||
StoreInst *Store1 = cast<StoreInst>(Inst);
|
||||
|
||||
AliasAnalysis::Location Loc0 = AA->getLocation(Store0);
|
||||
AliasAnalysis::Location Loc1 = AA->getLocation(Store1);
|
||||
AliasAnalysis::Location Loc0 = MemoryLocation::get(Store0);
|
||||
AliasAnalysis::Location Loc1 = MemoryLocation::get(Store1);
|
||||
if (AA->isMustAlias(Loc0, Loc1) && Store0->isSameOperationAs(Store1) &&
|
||||
!isStoreSinkBarrierInRange(*(std::next(BasicBlock::iterator(Store1))),
|
||||
BB1->back(), Loc1) &&
|
||||
|
Reference in New Issue
Block a user