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

[SSAUpdaterBulk] Use SmallVector instead of SmallPtrSet for uses. 

llvm-svn: 330398
This commit is contained in:
Michael Zolotukhin 2018-04-20 07:56:00 +00:00
parent 4557178061
commit 79cb54b2d9
2 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
llvm/include/llvm/Transforms/Utils/SSAUpdaterBulk.h
View File

@ -41,7 +41,7 @@ class DominatorTree;
class SSAUpdaterBulk { class SSAUpdaterBulk {
struct RewriteInfo { struct RewriteInfo {
DenseMap<BasicBlock *, Value *> Defines; DenseMap<BasicBlock *, Value *> Defines;
SmallPtrSet<Use *, 4> Uses; SmallVector<Use *, 4> Uses;
StringRef Name; StringRef Name;
Type *Ty; Type *Ty;
RewriteInfo(){}; RewriteInfo(){};

7
llvm/lib/Transforms/Utils/SSAUpdaterBulk.cpp
View File

@ -61,7 +61,7 @@ void SSAUpdaterBulk::AddUse(unsigned Var, Use *U) {
assert(Rewrites.find(Var) != Rewrites.end() && "Should add variable first!"); assert(Rewrites.find(Var) != Rewrites.end() && "Should add variable first!");
DEBUG(dbgs() << "SSAUpdater: Var=" << Var << ": added a use" << *U->get() DEBUG(dbgs() << "SSAUpdater: Var=" << Var << ": added a use" << *U->get()
<< " in " << getUserBB(U)->getName() << "\n"); << " in " << getUserBB(U)->getName() << "\n");
Rewrites[Var].Uses.insert(U); Rewrites[Var].Uses.push_back(U);
} }
/// Return true if the SSAUpdater already has a value for the specified variable /// Return true if the SSAUpdater already has a value for the specified variable
@ -171,7 +171,10 @@ void SSAUpdaterBulk::RewriteAllUses(DominatorTree *DT,
} }
// Rewrite actual uses with the inserted definitions. // Rewrite actual uses with the inserted definitions.
for (auto U : R.Uses) { SmallPtrSet<Use *, 4> ProcessedUses;
for (Use *U : R.Uses) {
if (!ProcessedUses.insert(U).second)
continue;
Value *V = computeValueAt(getUserBB(U), R, DT); Value *V = computeValueAt(getUserBB(U), R, DT);
Value *OldVal = U->get(); Value *OldVal = U->get();
// Notify that users of the existing value that it is being replaced. // Notify that users of the existing value that it is being replaced.