aboutsummaryrefslogtreecommitdiff
path: root/lib/Transforms/Utils/PromoteMemoryToRegister.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'lib/Transforms/Utils/PromoteMemoryToRegister.cpp')
-rw-r--r--lib/Transforms/Utils/PromoteMemoryToRegister.cpp17
1 files changed, 11 insertions, 6 deletions
diff --git a/lib/Transforms/Utils/PromoteMemoryToRegister.cpp b/lib/Transforms/Utils/PromoteMemoryToRegister.cpp
index 1efa8c2393..6ea8947af9 100644
--- a/lib/Transforms/Utils/PromoteMemoryToRegister.cpp
+++ b/lib/Transforms/Utils/PromoteMemoryToRegister.cpp
@@ -28,7 +28,10 @@
/// isAllocaPromotable - Return true if this alloca is legal for promotion.
/// This is true if there are only loads and stores to the alloca...
///
-bool isAllocaPromotable(const AllocaInst *AI) {
+bool isAllocaPromotable(const AllocaInst *AI, const TargetData &TD) {
+ // FIXME: If the memory unit is of pointer or integer type, we can permit
+ // assignments to subsections of the memory unit.
+
// Only allow direct loads and stores...
for (Value::use_const_iterator UI = AI->use_begin(), UE = AI->use_end();
UI != UE; ++UI) // Loop over all of the uses of the alloca
@@ -48,6 +51,7 @@ namespace {
struct PromoteMem2Reg {
const std::vector<AllocaInst*> &Allocas; // the alloca instructions..
DominanceFrontier &DF;
+ const TargetData &TD;
std::map<Instruction*, unsigned> AllocaLookup; // reverse mapping of above
@@ -60,8 +64,9 @@ namespace {
std::vector<PHINode*> > NewPhiNodes; // the PhiNodes we're adding
public:
- PromoteMem2Reg(const std::vector<AllocaInst*> &A, DominanceFrontier &df)
- :Allocas(A), DF(df) {}
+ PromoteMem2Reg(const std::vector<AllocaInst*> &A, DominanceFrontier &df,
+ const TargetData &td)
+ : Allocas(A), DF(df), TD(td) {}
void run();
@@ -81,7 +86,7 @@ void PromoteMem2Reg::run() {
Function &F = *DF.getRoot()->getParent();
for (unsigned i = 0, e = Allocas.size(); i != e; ++i) {
- assert(isAllocaPromotable(Allocas[i]) &&
+ assert(isAllocaPromotable(Allocas[i], TD) &&
"Cannot promote non-promotable alloca!");
assert(Allocas[i]->getParent()->getParent() == &F &&
"All allocas should be in the same function, which is same as DF!");
@@ -240,6 +245,6 @@ void PromoteMem2Reg::RenamePass(BasicBlock *BB, BasicBlock *Pred,
/// of the function at all. All allocas must be from the same function.
///
void PromoteMemToReg(const std::vector<AllocaInst*> &Allocas,
- DominanceFrontier &DF) {
- PromoteMem2Reg(Allocas, DF).run();
+ DominanceFrontier &DF, const TargetData &TD) {
+ PromoteMem2Reg(Allocas, DF, TD).run();
}