[Hexagon] Avoid repeated map lookups (NFC) (#126185)

kazutakahirata · web-flow · commit 46f5662ebc46 · 2025-02-07T07:58:49.000-08:00
diff --git a/llvm/lib/Target/Hexagon/HexagonMachineFunctionInfo.h b/llvm/lib/Target/Hexagon/HexagonMachineFunctionInfo.h
@@ -70,12 +70,12 @@ class HexagonMachineFunctionInfo : public MachineFunctionInfo {
     PacketInfo[MI] |= Hexagon::EndPacket;
   }
   bool isStartPacket(const MachineInstr* MI) const {
-    return (PacketInfo.count(MI) &&
-            (PacketInfo.find(MI)->second & Hexagon::StartPacket));
+    auto It = PacketInfo.find(MI);
+    return It != PacketInfo.end() && (It->second & Hexagon::StartPacket);
   }
   bool isEndPacket(const MachineInstr* MI) const {
-    return (PacketInfo.count(MI) &&
-            (PacketInfo.find(MI)->second & Hexagon::EndPacket));
+    auto It = PacketInfo.find(MI);
+    return It != PacketInfo.end() && (It->second & Hexagon::EndPacket);
   }
   void setHasClobberLR(bool v) { HasClobberLR = v;  }
   bool hasClobberLR() const { return HasClobberLR; }

Original file line number	Diff line number	Diff line change
`@@ -70,12 +70,12 @@ class HexagonMachineFunctionInfo : public MachineFunctionInfo {`
`70`	`70`	`PacketInfo[MI] \|= Hexagon::EndPacket;`
`71`	`71`	`}`
`72`	`72`	`bool isStartPacket(const MachineInstr* MI) const {`
`73`		`- return (PacketInfo.count(MI) &&`
`74`		`- (PacketInfo.find(MI)->second & Hexagon::StartPacket));`
	`73`	`+ auto It = PacketInfo.find(MI);`
	`74`	`+ return It != PacketInfo.end() && (It->second & Hexagon::StartPacket);`
`75`	`75`	`}`
`76`	`76`	`bool isEndPacket(const MachineInstr* MI) const {`
`77`		`- return (PacketInfo.count(MI) &&`
`78`		`- (PacketInfo.find(MI)->second & Hexagon::EndPacket));`
	`77`	`+ auto It = PacketInfo.find(MI);`
	`78`	`+ return It != PacketInfo.end() && (It->second & Hexagon::EndPacket);`
`79`	`79`	`}`
`80`	`80`	`void setHasClobberLR(bool v) { HasClobberLR = v; }`
`81`	`81`	`bool hasClobberLR() const { return HasClobberLR; }`