diff options
author | Sanjoy Das <sanjoy@playingwithpointers.com> | 2016-06-16 21:00:57 +0000 |
---|---|---|
committer | Sanjoy Das <sanjoy@playingwithpointers.com> | 2016-06-16 21:00:57 +0000 |
commit | 1ab2fad3638f24b0b8762e5e24c351e67299508e (patch) | |
tree | e4dfffbd5f429da4c4e8589b5102a3c69a283559 /llvm/lib/Transforms/Scalar/EarlyCSE.cpp | |
parent | 07c6521aed9ae565dd3ee71245bd159fb3355ef1 (diff) | |
download | bcm5719-llvm-1ab2fad3638f24b0b8762e5e24c351e67299508e.tar.gz bcm5719-llvm-1ab2fad3638f24b0b8762e5e24c351e67299508e.zip |
[EarlyCSE] Minor cosmetic NFC changes
- Avoid implicit conversion from pointer to bool
- Add a comment when passing in a boolean value
llvm-svn: 272955
Diffstat (limited to 'llvm/lib/Transforms/Scalar/EarlyCSE.cpp')
-rw-r--r-- | llvm/lib/Transforms/Scalar/EarlyCSE.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Scalar/EarlyCSE.cpp b/llvm/lib/Transforms/Scalar/EarlyCSE.cpp index 48253f62634..0e93b0a37b0 100644 --- a/llvm/lib/Transforms/Scalar/EarlyCSE.cpp +++ b/llvm/lib/Transforms/Scalar/EarlyCSE.cpp @@ -434,7 +434,7 @@ private: bool isInvariantLoad() const { if (auto *LI = dyn_cast<LoadInst>(Inst)) - return LI->getMetadata(LLVMContext::MD_invariant_load); + return LI->getMetadata(LLVMContext::MD_invariant_load) != nullptr; return false; } @@ -763,7 +763,7 @@ bool EarlyCSE::processNode(DomTreeNode *Node) { AvailableLoads.insert( MemInst.getPointerOperand(), LoadValue(Inst, CurrentGeneration, MemInst.getMatchingId(), - MemInst.isAtomic(), false)); + MemInst.isAtomic(), /*IsInvariant=*/false)); // Remember that this was the last unordered store we saw for DSE. We // don't yet handle DSE on ordered or volatile stores since we don't |