[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