Skip to content

Commit 4e0afa6

Browse files
committed
[MLIR] Apply clang-tidy fixes for misc-use-internal-linkage in OpDefinitionsGen.cpp (NFC)
1 parent abd2dc9 commit 4e0afa6

File tree

1 file changed

+17
-12
lines changed

1 file changed

+17
-12
lines changed

mlir/tools/tblgen-to-irdl/OpDefinitionsGen.cpp

Lines changed: 17 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ static llvm::cl::opt<std::string>
4040
selectedDialect("dialect", llvm::cl::desc("The dialect to gen for"),
4141
llvm::cl::cat(dialectGenCat), llvm::cl::Required);
4242

43-
Value createPredicate(OpBuilder &builder, tblgen::Pred pred) {
43+
static Value createPredicate(OpBuilder &builder, tblgen::Pred pred) {
4444
MLIRContext *ctx = builder.getContext();
4545

4646
if (pred.isCombined()) {
@@ -68,21 +68,22 @@ Value createPredicate(OpBuilder &builder, tblgen::Pred pred) {
6868
return op;
6969
}
7070

71-
Value typeToConstraint(OpBuilder &builder, Type type) {
71+
static Value typeToConstraint(OpBuilder &builder, Type type) {
7272
MLIRContext *ctx = builder.getContext();
7373
auto op =
7474
irdl::IsOp::create(builder, UnknownLoc::get(ctx), TypeAttr::get(type));
7575
return op.getOutput();
7676
}
7777

78-
Value baseToConstraint(OpBuilder &builder, StringRef baseClass) {
78+
static Value baseToConstraint(OpBuilder &builder, StringRef baseClass) {
7979
MLIRContext *ctx = builder.getContext();
8080
auto op = irdl::BaseOp::create(builder, UnknownLoc::get(ctx),
8181
StringAttr::get(ctx, baseClass));
8282
return op.getOutput();
8383
}
8484

85-
std::optional<Type> recordToType(MLIRContext *ctx, const Record &predRec) {
85+
static std::optional<Type> recordToType(MLIRContext *ctx,
86+
const Record &predRec) {
8687
if (predRec.isSubClassOf("I")) {
8788
auto width = predRec.getValueAsInt("bitwidth");
8889
return IntegerType::get(ctx, width, IntegerType::Signless);
@@ -171,7 +172,8 @@ std::optional<Type> recordToType(MLIRContext *ctx, const Record &predRec) {
171172
return std::nullopt;
172173
}
173174

174-
Value createTypeConstraint(OpBuilder &builder, tblgen::Constraint constraint) {
175+
static Value createTypeConstraint(OpBuilder &builder,
176+
tblgen::Constraint constraint) {
175177
MLIRContext *ctx = builder.getContext();
176178
const Record &predRec = constraint.getDef();
177179

@@ -260,7 +262,8 @@ Value createTypeConstraint(OpBuilder &builder, tblgen::Constraint constraint) {
260262
return createPredicate(builder, constraint.getPredicate());
261263
}
262264

263-
Value createAttrConstraint(OpBuilder &builder, tblgen::Constraint constraint) {
265+
static Value createAttrConstraint(OpBuilder &builder,
266+
tblgen::Constraint constraint) {
264267
MLIRContext *ctx = builder.getContext();
265268
const Record &predRec = constraint.getDef();
266269

@@ -341,7 +344,8 @@ Value createAttrConstraint(OpBuilder &builder, tblgen::Constraint constraint) {
341344
return createPredicate(builder, constraint.getPredicate());
342345
}
343346

344-
Value createRegionConstraint(OpBuilder &builder, tblgen::Region constraint) {
347+
static Value createRegionConstraint(OpBuilder &builder,
348+
tblgen::Region constraint) {
345349
MLIRContext *ctx = builder.getContext();
346350
const Record &predRec = constraint.getDef();
347351

@@ -383,8 +387,8 @@ static StringRef getAttrName(tblgen::AttrDef &tblgenType) {
383387
}
384388

385389
/// Extract an operation to IRDL.
386-
irdl::OperationOp createIRDLOperation(OpBuilder &builder,
387-
tblgen::Operator &tblgenOp) {
390+
static irdl::OperationOp createIRDLOperation(OpBuilder &builder,
391+
tblgen::Operator &tblgenOp) {
388392
MLIRContext *ctx = builder.getContext();
389393
StringRef opName = getOperatorName(tblgenOp);
390394

@@ -488,7 +492,8 @@ irdl::OperationOp createIRDLOperation(OpBuilder &builder,
488492
return op;
489493
}
490494

491-
irdl::TypeOp createIRDLType(OpBuilder &builder, tblgen::TypeDef &tblgenType) {
495+
static irdl::TypeOp createIRDLType(OpBuilder &builder,
496+
tblgen::TypeDef &tblgenType) {
492497
MLIRContext *ctx = builder.getContext();
493498
StringRef typeName = getTypeName(tblgenType);
494499
std::string combined = ("!" + typeName).str();
@@ -501,8 +506,8 @@ irdl::TypeOp createIRDLType(OpBuilder &builder, tblgen::TypeDef &tblgenType) {
501506
return op;
502507
}
503508

504-
irdl::AttributeOp createIRDLAttr(OpBuilder &builder,
505-
tblgen::AttrDef &tblgenAttr) {
509+
static irdl::AttributeOp createIRDLAttr(OpBuilder &builder,
510+
tblgen::AttrDef &tblgenAttr) {
506511
MLIRContext *ctx = builder.getContext();
507512
StringRef attrName = getAttrName(tblgenAttr);
508513
std::string combined = ("#" + attrName).str();

0 commit comments

Comments
 (0)