diff options
author | Arnaud A. de Grandmaison <arnaud.degrandmaison@arm.com> | 2015-10-09 09:23:01 +0000 |
---|---|---|
committer | Arnaud A. de Grandmaison <arnaud.degrandmaison@arm.com> | 2015-10-09 09:23:01 +0000 |
commit | 859b2ac07d0ce71e3c6ecae6a9e21d87f58d55b8 (patch) | |
tree | 744456a6346d27c60f8f40812adcab941c04c409 /llvm/lib/Transforms/Scalar | |
parent | 5b3bab40b2341e7b6c81e50d3b416515203a1e7f (diff) | |
download | bcm5719-llvm-859b2ac07d0ce71e3c6ecae6a9e21d87f58d55b8.tar.gz bcm5719-llvm-859b2ac07d0ce71e3c6ecae6a9e21d87f58d55b8.zip |
[EarlyCSE] Address post commit review for r249523.
llvm-svn: 249814
Diffstat (limited to 'llvm/lib/Transforms/Scalar')
-rw-r--r-- | llvm/lib/Transforms/Scalar/EarlyCSE.cpp | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/llvm/lib/Transforms/Scalar/EarlyCSE.cpp b/llvm/lib/Transforms/Scalar/EarlyCSE.cpp index 008b0e5ffb3..4dab49372e3 100644 --- a/llvm/lib/Transforms/Scalar/EarlyCSE.cpp +++ b/llvm/lib/Transforms/Scalar/EarlyCSE.cpp @@ -291,12 +291,12 @@ public: /// after every possibly writing memory operation, which ensures that we only /// CSE loads with other loads that have no intervening store. struct LoadValue { - Value *data; - unsigned generation; - int matchingId; - LoadValue() : data(nullptr), generation(0), matchingId(-1) {} - LoadValue(Value *data, unsigned generation, unsigned matchingId) - : data(data), generation(generation), matchingId(matchingId) {} + Value *Data; + unsigned Generation; + int MatchingId; + LoadValue() : Data(nullptr), Generation(0), MatchingId(-1) {} + LoadValue(Value *Data, unsigned Generation, unsigned MatchingId) + : Data(Data), Generation(Generation), MatchingId(MatchingId) {} }; typedef RecyclingAllocator<BumpPtrAllocator, ScopedHashTableVal<Value *, LoadValue>> @@ -568,12 +568,12 @@ bool EarlyCSE::processNode(DomTreeNode *Node) { // If we have an available version of this load, and if it is the right // generation, replace this instruction. LoadValue InVal = AvailableLoads.lookup(MemInst.getPtr()); - if (InVal.data != nullptr && InVal.generation == CurrentGeneration && - InVal.matchingId == MemInst.getMatchingId()) { - Value *Op = getOrCreateResult(InVal.data, Inst->getType()); + if (InVal.Data != nullptr && InVal.Generation == CurrentGeneration && + InVal.MatchingId == MemInst.getMatchingId()) { + Value *Op = getOrCreateResult(InVal.Data, Inst->getType()); if (Op != nullptr) { DEBUG(dbgs() << "EarlyCSE CSE LOAD: " << *Inst - << " to: " << *InVal.data << '\n'); + << " to: " << *InVal.Data << '\n'); if (!Inst->use_empty()) Inst->replaceAllUsesWith(Op); Inst->eraseFromParent(); |