[llvm] 86d9faa - [MachineSink] Use LLVM ADTs (NFC) (#68677)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 12 02:04:46 PDT 2023
Author: Momchil Velikov
Date: 2023-10-12T10:04:41+01:00
New Revision: 86d9faa5a97610a1b50b5bffe502b3807c03b0d9
URL: https://github.com/llvm/llvm-project/commit/86d9faa5a97610a1b50b5bffe502b3807c03b0d9
DIFF: https://github.com/llvm/llvm-project/commit/86d9faa5a97610a1b50b5bffe502b3807c03b0d9.diff
LOG: [MachineSink] Use LLVM ADTs (NFC) (#68677)
Replace a few uses of `std::map` with `llvm::DenseMap`.
Added:
Modified:
llvm/lib/CodeGen/MachineSink.cpp
Removed:
################################################################################
diff --git a/llvm/lib/CodeGen/MachineSink.cpp b/llvm/lib/CodeGen/MachineSink.cpp
index 40f33d664414dbb..9a0d1cc94d24b3a 100644
--- a/llvm/lib/CodeGen/MachineSink.cpp
+++ b/llvm/lib/CodeGen/MachineSink.cpp
@@ -57,7 +57,6 @@
#include <algorithm>
#include <cassert>
#include <cstdint>
-#include <map>
#include <utility>
#include <vector>
@@ -139,7 +138,7 @@ namespace {
DenseSet<Register> RegsToClearKillFlags;
using AllSuccsCache =
- std::map<MachineBasicBlock *, SmallVector<MachineBasicBlock *, 4>>;
+ DenseMap<MachineBasicBlock *, SmallVector<MachineBasicBlock *, 4>>;
/// DBG_VALUE pointer and flag. The flag is true if this DBG_VALUE is
/// post-dominated by another DBG_VALUE of the same variable location.
@@ -160,14 +159,15 @@ namespace {
/// current block.
DenseSet<DebugVariable> SeenDbgVars;
- std::map<std::pair<MachineBasicBlock *, MachineBasicBlock *>, bool>
+ DenseMap<std::pair<MachineBasicBlock *, MachineBasicBlock *>, bool>
HasStoreCache;
- std::map<std::pair<MachineBasicBlock *, MachineBasicBlock *>,
- std::vector<MachineInstr *>>
+
+ DenseMap<std::pair<MachineBasicBlock *, MachineBasicBlock *>,
+ SmallVector<MachineInstr *>>
StoreInstrCache;
/// Cached BB's register pressure.
- std::map<const MachineBasicBlock *, std::vector<unsigned>>
+ DenseMap<const MachineBasicBlock *, std::vector<unsigned>>
CachedRegisterPressure;
bool EnableSinkAndFold;
@@ -1429,11 +1429,11 @@ bool MachineSinking::hasStoreBetween(MachineBasicBlock *From,
// Does these two blocks pair be queried before and have a definite cached
// result?
- if (HasStoreCache.find(BlockPair) != HasStoreCache.end())
- return HasStoreCache[BlockPair];
+ if (auto It = HasStoreCache.find(BlockPair); It != HasStoreCache.end())
+ return It->second;
- if (StoreInstrCache.find(BlockPair) != StoreInstrCache.end())
- return llvm::any_of(StoreInstrCache[BlockPair], [&](MachineInstr *I) {
+ if (auto It = StoreInstrCache.find(BlockPair); It != StoreInstrCache.end())
+ return llvm::any_of(It->second, [&](MachineInstr *I) {
return I->mayAlias(AA, MI, false);
});
More information about the llvm-commits
mailing list