diff options
| author | Davide Italiano <davide@freebsd.org> | 2017-05-12 15:22:45 +0000 |
|---|---|---|
| committer | Davide Italiano <davide@freebsd.org> | 2017-05-12 15:22:45 +0000 |
| commit | 41f5c7bcba78268607b94998ce0608fe2ed2bde0 (patch) | |
| tree | d73bf5e4d86be977e613d9eed15a0a276bd7dee2 /llvm/lib/Transforms/Scalar | |
| parent | fdcab2cbf2ef5bb753fc8542eebbaf9251dccbd6 (diff) | |
| download | bcm5719-llvm-41f5c7bcba78268607b94998ce0608fe2ed2bde0.tar.gz bcm5719-llvm-41f5c7bcba78268607b94998ce0608fe2ed2bde0.zip | |
[NewGVN] Don't incorrectly reset the memory leader.
This code was missing a check for stores, so we were thinking the
congruency class didn't have any memory members, and reset the
memory leader.
Differential Revision: https://reviews.llvm.org/D33056
llvm-svn: 302905
Diffstat (limited to 'llvm/lib/Transforms/Scalar')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/NewGVN.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Scalar/NewGVN.cpp b/llvm/lib/Transforms/Scalar/NewGVN.cpp index 0c2a30953e7..e0f6af92ebb 100644 --- a/llvm/lib/Transforms/Scalar/NewGVN.cpp +++ b/llvm/lib/Transforms/Scalar/NewGVN.cpp @@ -1409,7 +1409,7 @@ bool NewGVN::setMemoryClass(const MemoryAccess *From, NewClass->memory_insert(MP); // This may have killed the class if it had no non-memory members if (OldClass->getMemoryLeader() == From) { - if (OldClass->memory_empty()) { + if (OldClass->definesNoMemory()) { OldClass->setMemoryLeader(nullptr); } else { OldClass->setMemoryLeader(getNextMemoryLeader(OldClass)); |
