[llvm] 9ddc043 - [TableGen] Use structured binding in one more place (NFC)

Sergei Barannikov via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 17 20:15:54 PDT 2025


Author: Sergei Barannikov
Date: 2025-08-18T06:15:44+03:00
New Revision: 9ddc043538329a0a1386d12e191de7b4c43051e9

URL: https://github.com/llvm/llvm-project/commit/9ddc043538329a0a1386d12e191de7b4c43051e9
DIFF: https://github.com/llvm/llvm-project/commit/9ddc043538329a0a1386d12e191de7b4c43051e9.diff

LOG: [TableGen] Use structured binding in one more place (NFC)

Added: 
    

Modified: 
    llvm/utils/TableGen/DecoderEmitter.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/utils/TableGen/DecoderEmitter.cpp b/llvm/utils/TableGen/DecoderEmitter.cpp
index b01a79e37ce48..f69e47e49c1eb 100644
--- a/llvm/utils/TableGen/DecoderEmitter.cpp
+++ b/llvm/utils/TableGen/DecoderEmitter.cpp
@@ -2400,17 +2400,17 @@ static void collectHwModesReferencedForEncodings(
     NamespacesHwModesMap &NamespacesWithHwModes) {
   SmallBitVector BV(HWM.getNumModeIds());
   for (const auto &MS : HWM.getHwModeSelects()) {
-    for (const HwModeSelect::PairType &P : MS.second.Items) {
-      if (P.second->isSubClassOf("InstructionEncoding")) {
+    for (auto [HwModeID, EncodingDef] : MS.second.Items) {
+      if (EncodingDef->isSubClassOf("InstructionEncoding")) {
         std::string DecoderNamespace =
-            P.second->getValueAsString("DecoderNamespace").str();
-        if (P.first == DefaultMode) {
+            EncodingDef->getValueAsString("DecoderNamespace").str();
+        if (HwModeID == DefaultMode) {
           NamespacesWithHwModes[DecoderNamespace].insert("");
         } else {
           NamespacesWithHwModes[DecoderNamespace].insert(
-              HWM.getMode(P.first).Name);
+              HWM.getMode(HwModeID).Name);
         }
-        BV.set(P.first);
+        BV.set(HwModeID);
       }
     }
   }
@@ -2494,13 +2494,13 @@ namespace {
     const Record *InstDef = NumberedInstruction->TheDef;
     if (const Record *RV = InstDef->getValueAsOptionalDef("EncodingInfos")) {
       EncodingInfoByHwMode EBM(RV, HWM);
-      for (auto &[ModeId, Encoding] : EBM) {
+      for (auto [HwModeID, EncodingDef] : EBM) {
         // DecoderTables with DefaultMode should not have any suffix.
-        if (ModeId == DefaultMode) {
-          NumberedEncodings.emplace_back(Encoding, NumberedInstruction, "");
+        if (HwModeID == DefaultMode) {
+          NumberedEncodings.emplace_back(EncodingDef, NumberedInstruction, "");
         } else {
-          NumberedEncodings.emplace_back(Encoding, NumberedInstruction,
-                                         HWM.getMode(ModeId).Name);
+          NumberedEncodings.emplace_back(EncodingDef, NumberedInstruction,
+                                         HWM.getMode(HwModeID).Name);
         }
       }
       continue;


        


More information about the llvm-commits mailing list