[llvm] [TableGen][DecoderEmitter] Change SmallSetVector to SetVector (NFC) (PR #159108)
Sergei Barannikov via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 16 08:06:14 PDT 2025
https://github.com/s-barannikov created https://github.com/llvm/llvm-project/pull/159108
SmallSetVector is too optimistic, there are usually more than 16 unique decoders and predicates. Modernize `typedef` to `using` while here.
>From b5b2b887dd36455d7b2bcf31b866f1ff447b3f17 Mon Sep 17 00:00:00 2001
From: Sergei Barannikov <barannikov88 at gmail.com>
Date: Tue, 16 Sep 2025 18:05:33 +0300
Subject: [PATCH] [TableGen][DecoderEmitter] Change SmallSetVector to SetVector
(NFC)
SmallSetVector is too optimistic, there are usually more than 16 unique
decoders and predicates. Modernize `typedef` to `using` while here.
---
llvm/utils/TableGen/DecoderEmitter.cpp | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/llvm/utils/TableGen/DecoderEmitter.cpp b/llvm/utils/TableGen/DecoderEmitter.cpp
index 4b04b33502d25..77bf60e14c9a8 100644
--- a/llvm/utils/TableGen/DecoderEmitter.cpp
+++ b/llvm/utils/TableGen/DecoderEmitter.cpp
@@ -160,8 +160,8 @@ class LessEncodingIDByWidth {
}
};
-typedef SmallSetVector<CachedHashString, 16> PredicateSet;
-typedef SmallSetVector<CachedHashString, 16> DecoderSet;
+using PredicateSet = SetVector<CachedHashString>;
+using DecoderSet = SetVector<CachedHashString>;
class DecoderTable {
public:
@@ -274,7 +274,7 @@ class DecoderEmitter {
void emitInstrLenTable(formatted_raw_ostream &OS,
ArrayRef<unsigned> InstrLen) const;
void emitPredicateFunction(formatted_raw_ostream &OS,
- PredicateSet &Predicates) const;
+ const PredicateSet &Predicates) const;
void emitDecoderFunction(formatted_raw_ostream &OS,
const DecoderSet &Decoders,
unsigned BucketBitWidth) const;
@@ -838,8 +838,8 @@ void DecoderEmitter::emitInstrLenTable(formatted_raw_ostream &OS,
OS << "};\n\n";
}
-void DecoderEmitter::emitPredicateFunction(formatted_raw_ostream &OS,
- PredicateSet &Predicates) const {
+void DecoderEmitter::emitPredicateFunction(
+ formatted_raw_ostream &OS, const PredicateSet &Predicates) const {
// The predicate function is just a big switch statement based on the
// input predicate index.
OS << "static bool checkDecoderPredicate(unsigned Idx, const FeatureBitset "
More information about the llvm-commits
mailing list