maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[GVNHoist] Remove a home-grown version of replaceUsesOfWith 

replaceUsesOfWith will, on average, consider fewer values when trying
to do the replacement.

No functional change is intended.

llvm-svn: 275884
This commit is contained in:
David Majnemer 2016-07-18 19:14:14 +00:00
parent 7a3040dc83
commit 04854ab1e5
1 changed files with 2 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
llvm/lib/Transforms/Scalar/GVNHoist.cpp
View File

@ -574,19 +574,6 @@ public:
llvm_unreachable("Both I and J must be from same BB");
}
// Replace the use of From with To in Insn.
void replaceUseWith(Instruction *Insn, Value *From, Value *To) const {
for (Value::use_iterator UI = From->use_begin(), UE = From->use_end();
UI != UE;) {
Use &U = *UI++;
if (U.getUser() == Insn) {
U.set(To);
return;
}
}
llvm_unreachable("should replace exactly once");
}
bool makeOperandsAvailable(Instruction *Repl, BasicBlock *HoistPt) const {
// Check whether the GEP of a ld/st can be synthesized at HoistPt.
Instruction *Gep = nullptr;
@ -612,13 +599,13 @@ public:
// Copy the gep before moving the ld/st.
Instruction *ClonedGep = Gep->clone();
ClonedGep->insertBefore(HoistPt->getTerminator());
replaceUseWith(Repl, Gep, ClonedGep);
Repl->replaceUsesOfWith(Gep, ClonedGep);
// Also copy Val.
if (Val) {
Instruction *ClonedVal = Val->clone();
ClonedVal->insertBefore(HoistPt->getTerminator());
replaceUseWith(Repl, Val, ClonedVal);
Repl->replaceUsesOfWith(Val, ClonedVal);
}
return true;