[llvm] [GlobalISel] Avoid repeated hash lookups (NFC) (PR #127372)

via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 15 20:49:56 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-llvm-globalisel

Author: Kazu Hirata (kazutakahirata)

<details>
<summary>Changes</summary>



---
Full diff: https://github.com/llvm/llvm-project/pull/127372.diff


1 Files Affected:

- (modified) llvm/lib/CodeGen/GlobalISel/LegacyLegalizerInfo.cpp (+4-4) 


``````````diff
diff --git a/llvm/lib/CodeGen/GlobalISel/LegacyLegalizerInfo.cpp b/llvm/lib/CodeGen/GlobalISel/LegacyLegalizerInfo.cpp
index 45b403bdd0765..f338f66997657 100644
--- a/llvm/lib/CodeGen/GlobalISel/LegacyLegalizerInfo.cpp
+++ b/llvm/lib/CodeGen/GlobalISel/LegacyLegalizerInfo.cpp
@@ -305,10 +305,10 @@ LegacyLegalizerInfo::findScalarLegalAction(const InstrAspect &Aspect) const {
   if (Aspect.Opcode < FirstOp || Aspect.Opcode > LastOp)
     return {NotFound, LLT()};
   const unsigned OpcodeIdx = getOpcodeIdxForOpcode(Aspect.Opcode);
-  if (Aspect.Type.isPointer() &&
-      AddrSpace2PointerActions[OpcodeIdx].find(Aspect.Type.getAddressSpace()) ==
-          AddrSpace2PointerActions[OpcodeIdx].end()) {
-    return {NotFound, LLT()};
+  if (Aspect.Type.isPointer()) {
+    auto &PA = AddrSpace2PointerActions[OpcodeIdx];
+    if (PA.find(Aspect.Type.getAddressSpace()) == PA.end())
+      return {NotFound, LLT()};
   }
   const SmallVector<SizeAndActionsVec, 1> &Actions =
       Aspect.Type.isPointer()

``````````

</details>


https://github.com/llvm/llvm-project/pull/127372


More information about the llvm-commits mailing list