[llvm] [Target] Use DenseSet instead of DenseMap (NFC) (PR #132619)
via llvm-commits
llvm-commits at lists.llvm.org
Sun Mar 23 09:51:17 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-x86
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
This patch uses DenseSet instead of DenseMap. Note that the set of
Registers that map to true without this patch is the same as the set
of Registers that are present in the set with this patch. This patch
is inspired by:
commit d7879e524fbbc4c2790dac62343444191f736f00
Author: Craig Topper <craig.topper@<!-- -->sifive.com>
Date: Wed Mar 19 08:32:09 2025 -0700
---
Full diff: https://github.com/llvm/llvm-project/pull/132619.diff
2 Files Affected:
- (modified) llvm/lib/Target/VE/VEISelLowering.cpp (+3-3)
- (modified) llvm/lib/Target/X86/X86ISelLowering.cpp (+3-3)
``````````diff
diff --git a/llvm/lib/Target/VE/VEISelLowering.cpp b/llvm/lib/Target/VE/VEISelLowering.cpp
index b7fa089921c2d..313c894cafa85 100644
--- a/llvm/lib/Target/VE/VEISelLowering.cpp
+++ b/llvm/lib/Target/VE/VEISelLowering.cpp
@@ -2645,15 +2645,15 @@ VETargetLowering::emitSjLjDispatchBlock(MachineInstr &MI,
if (!II.isCall())
continue;
- DenseMap<Register, bool> DefRegs;
+ DenseSet<Register> DefRegs;
for (auto &MOp : II.operands())
if (MOp.isReg())
- DefRegs[MOp.getReg()] = true;
+ DefRegs.insert(MOp.getReg());
MachineInstrBuilder MIB(*MF, &II);
for (unsigned RI = 0; SavedRegs[RI]; ++RI) {
Register Reg = SavedRegs[RI];
- if (!DefRegs[Reg])
+ if (!DefRegs.contains(Reg))
MIB.addReg(Reg, RegState::ImplicitDefine | RegState::Dead);
}
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp
index a7db154bfaa3b..0f737b1f8d854 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -37430,15 +37430,15 @@ X86TargetLowering::EmitSjLjDispatchBlock(MachineInstr &MI,
if (!II.isCall())
continue;
- DenseMap<Register, bool> DefRegs;
+ DenseSet<Register> DefRegs;
for (auto &MOp : II.operands())
if (MOp.isReg())
- DefRegs[MOp.getReg()] = true;
+ DefRegs.insert(MOp.getReg());
MachineInstrBuilder MIB(*MF, &II);
for (unsigned RegIdx = 0; SavedRegs[RegIdx]; ++RegIdx) {
Register Reg = SavedRegs[RegIdx];
- if (!DefRegs[Reg])
+ if (!DefRegs.contains(Reg))
MIB.addReg(Reg, RegState::ImplicitDefine | RegState::Dead);
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/132619
More information about the llvm-commits
mailing list