[llvm] [SPIRV] Avoid repeated map lookups. NFC (PR #125036)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 29 22:50:07 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-spir-v
Author: Craig Topper (topperc)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/125036.diff
1 Files Affected:
- (modified) llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.h (+5-3)
``````````diff
diff --git a/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.h b/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.h
index 79b5444cca205da..e6b8a381b718006 100644
--- a/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.h
+++ b/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.h
@@ -180,11 +180,13 @@ struct ModuleAnalysisInfo {
if (RI == RegisterAliasTable[MF].end()) {
return Register(0);
}
- return RegisterAliasTable[MF][Reg];
+ return RI->second;
}
bool hasRegisterAlias(const MachineFunction *MF, Register Reg) {
- return RegisterAliasTable.find(MF) != RegisterAliasTable.end() &&
- RegisterAliasTable[MF].find(Reg) != RegisterAliasTable[MF].end();
+ auto RI = RegisterAliasTable.find(MF);
+ if (RI == RegisterAliasTable.end())
+ return false;
+ return RI->second.find(Reg) != RI->second.end();
}
unsigned getNextID() { return MaxID++; }
bool hasMBBRegister(const MachineBasicBlock &MBB) {
``````````
</details>
https://github.com/llvm/llvm-project/pull/125036
More information about the llvm-commits
mailing list