Fix incorrect computation of mod/ref sets. Do not ask for mod/ref information

for objects of size 0.

llvm-svn: 14908
This commit is contained in:
Chris Lattner 2004-07-17 07:40:34 +00:00
parent 2e8690bf57
commit 3bbaaaa940
1 changed files with 29 additions and 21 deletions

View File

@ -21,7 +21,7 @@
#include "llvm/iOther.h"
#include "llvm/iTerminators.h"
#include "llvm/Pass.h"
#include "llvm/Type.h"
#include "llvm/DerivedTypes.h"
#include "llvm/Analysis/AliasAnalysis.h"
#include "llvm/Assembly/Writer.h"
#include "llvm/Support/InstIterator.h"
@ -137,27 +137,35 @@ bool AAEval::runOnFunction(Function &F) {
// Mod/ref alias analysis: compare all pairs of calls and values
for (std::set<Value *>::iterator V = Pointers.begin(), Ve = Pointers.end();
V != Ve; ++V)
for (std::set<CallSite>::iterator C = CallSites.begin(),
Ce = CallSites.end(); C != Ce; ++C) {
Instruction *I = C->getInstruction();
switch (AA.getModRefInfo(*C, *V, (*V)->getType()->getPrimitiveSize())) {
case AliasAnalysis::NoModRef:
PrintModRefResults("NoModRef", PrintNoModRef, I, *V, F.getParent());
++NoModRef; break;
case AliasAnalysis::Mod:
PrintModRefResults(" Mod", PrintMod, I, *V, F.getParent());
++Mod; break;
case AliasAnalysis::Ref:
PrintModRefResults(" Ref", PrintRef, I, *V, F.getParent());
++Ref; break;
case AliasAnalysis::ModRef:
PrintModRefResults(" ModRef", PrintModRef, I, *V, F.getParent());
++ModRef; break;
default:
std::cerr << "Unknown alias query result!\n";
V != Ve; ++V) {
unsigned Size = 0;
if (const PointerType *PTy = dyn_cast<PointerType>((*V)->getType()))
if (!(Size = PTy->getElementType()->getPrimitiveSize()))
if (isa<PointerType>(PTy->getElementType()))
Size = 4; // This is a hack, but it's good enough for eval.
if (Size)
for (std::set<CallSite>::iterator C = CallSites.begin(),
Ce = CallSites.end(); C != Ce; ++C) {
Instruction *I = C->getInstruction();
switch (AA.getModRefInfo(*C, *V, Size)) {
case AliasAnalysis::NoModRef:
PrintModRefResults("NoModRef", PrintNoModRef, I, *V, F.getParent());
++NoModRef; break;
case AliasAnalysis::Mod:
PrintModRefResults(" Mod", PrintMod, I, *V, F.getParent());
++Mod; break;
case AliasAnalysis::Ref:
PrintModRefResults(" Ref", PrintRef, I, *V, F.getParent());
++Ref; break;
case AliasAnalysis::ModRef:
PrintModRefResults(" ModRef", PrintModRef, I, *V, F.getParent());
++ModRef; break;
default:
std::cerr << "Unknown alias query result!\n";
}
}
}
}
return false;
}