diff --git a/lib/Analysis/DataStructure/DataStructureAA.cpp b/lib/Analysis/DataStructure/DataStructureAA.cpp index 1778574f46e..5c3d946478a 100644 --- a/lib/Analysis/DataStructure/DataStructureAA.cpp +++ b/lib/Analysis/DataStructure/DataStructureAA.cpp @@ -24,11 +24,13 @@ namespace { // program. // bool run(Module &M) { + InitializeAliasAnalysis(this); TD = &getAnalysis(); return false; } virtual void getAnalysisUsage(AnalysisUsage &AU) const { + AliasAnalysis::getAnalysisUsage(AU); AU.setPreservesAll(); // Does not transform code... AU.addRequired(); // Uses TD Datastructures AU.addRequired(); // Chains to another AA impl... @@ -39,19 +41,8 @@ namespace { // // alias - This is the only method here that does anything interesting... - Result alias(const Value *V1, const Value *V2); - - /// canCallModify - Not implemented yet: FIXME - /// - Result canCallModify(const CallInst &CI, const Value *Ptr) { - return MayAlias; - } - - /// canInvokeModify - Not implemented yet: FIXME - /// - Result canInvokeModify(const InvokeInst &I, const Value *Ptr) { - return MayAlias; - } + AliasResult alias(const Value *V1, unsigned V1Size, + const Value *V2, unsigned V2Size); }; // Register the pass... @@ -75,7 +66,9 @@ static const Function *getValueFunction(const Value *V) { } // alias - This is the only method here that does anything interesting... -AliasAnalysis::Result DSAA::alias(const Value *V1, const Value *V2) { +AliasAnalysis::AliasResult DSAA::alias(const Value *V1, unsigned V1Size, + const Value *V2, unsigned V2Size) { + // FIXME: This should handle the Size argument as well! const Function *F1 = getValueFunction(V1); const Function *F2 = getValueFunction(V2); assert((!F1 || !F2 || F1 == F2) && "Alias query for 2 different functions?"); @@ -113,5 +106,5 @@ AliasAnalysis::Result DSAA::alias(const Value *V1, const Value *V2) { // FIXME: we could improve on this by checking the globals graph for aliased // global queries... - return getAnalysis().alias(V1, V2); + return getAnalysis().alias(V1, V1Size, V2, V2Size); } diff --git a/lib/Analysis/DataStructure/Steensgaard.cpp b/lib/Analysis/DataStructure/Steensgaard.cpp index 94b63d77a61..2f370258ff7 100644 --- a/lib/Analysis/DataStructure/Steensgaard.cpp +++ b/lib/Analysis/DataStructure/Steensgaard.cpp @@ -18,7 +18,7 @@ namespace { DSGraph *ResultGraph; DSGraph *GlobalsGraph; // FIXME: Eliminate globals graph stuff from DNE public: - Steens() : ResultGraph(0) {} + Steens() : ResultGraph(0), GlobalsGraph(0) {} ~Steens() { releaseMyMemory(); assert(ResultGraph == 0 && "releaseMemory not called?"); @@ -36,6 +36,7 @@ namespace { virtual void releaseMyMemory() { delete ResultGraph; ResultGraph = 0; } virtual void getAnalysisUsage(AnalysisUsage &AU) const { + AliasAnalysis::getAnalysisUsage(AU); AU.setPreservesAll(); // Does not transform code... AU.addRequired(); // Uses local dsgraph AU.addRequired(); // Chains to another AA impl... @@ -52,20 +53,9 @@ namespace { // // alias - This is the only method here that does anything interesting... - Result alias(const Value *V1, const Value *V2); + AliasResult alias(const Value *V1, unsigned V1Size, + const Value *V2, unsigned V2Size); - /// canCallModify - Not implemented yet: FIXME - /// - Result canCallModify(const CallInst &CI, const Value *Ptr) { - return MayAlias; - } - - /// canInvokeModify - Not implemented yet: FIXME - /// - Result canInvokeModify(const InvokeInst &I, const Value *Ptr) { - return MayAlias; - } - private: void ResolveFunctionCall(Function *F, const DSCallSite &Call, DSNodeHandle &RetVal); @@ -108,6 +98,7 @@ void Steens::ResolveFunctionCall(Function *F, const DSCallSite &Call, /// program. /// bool Steens::run(Module &M) { + InitializeAliasAnalysis(this); assert(ResultGraph == 0 && "Result graph already allocated!"); LocalDataStructures &LDS = getAnalysis(); @@ -212,7 +203,9 @@ bool Steens::run(Module &M) { } // alias - This is the only method here that does anything interesting... -AliasAnalysis::Result Steens::alias(const Value *V1, const Value *V2) { +AliasAnalysis::AliasResult Steens::alias(const Value *V1, unsigned V1Size, + const Value *V2, unsigned V2Size) { + // FIXME: HANDLE Size argument! assert(ResultGraph && "Result graph has not been computed yet!"); hash_map &GSM = ResultGraph->getScalarMap(); @@ -239,5 +232,5 @@ AliasAnalysis::Result Steens::alias(const Value *V1, const Value *V2) { // If we cannot determine alias properties based on our graph, fall back on // some other AA implementation. // - return getAnalysis().alias(V1, V2); + return getAnalysis().alias(V1, V1Size, V2, V2Size); }