[Codegen][nfc] Drop redundant public from struct inheritance. 3/5 (#23771)

Signed-off-by: hanhanW <[email protected]>

Author: hanhanW

compiler/src/iree/compiler/Codegen/Common/BubbleUpOrdinalOps.cpp

@@ -43,7 +43,7 @@ namespace {
 /// `iree_tensor_ext.dispatch.workload.ordinal` ops.
 template <typename CastOpTy>
 struct BubbleUpAcrossCastOp
-    : public OpRewritePattern<IREE::TensorExt::DispatchWorkloadOrdinalOp> {
+    : OpRewritePattern<IREE::TensorExt::DispatchWorkloadOrdinalOp> {
   using Base::Base;
   LogicalResult
   matchAndRewrite(IREE::TensorExt::DispatchWorkloadOrdinalOp ordinalOp,

compiler/src/iree/compiler/Codegen/Common/CPU/CPUPrepareUkernels.cpp

@@ -173,7 +173,7 @@ dropBatchTileSize(IREE::CPU::LoweringConfigAttr config) {
 
 /// Pattern to convert linalg.batch_mmt4d with batch dim = 1 into mmt4d.
 struct ConvertBatchMmt4DtoMmt4DPattern
-    : public OpRewritePattern<linalg::BatchMmt4DOp> {
+    : OpRewritePattern<linalg::BatchMmt4DOp> {
   using Base::Base;
 
   LogicalResult matchAndRewrite(linalg::BatchMmt4DOp op,
@@ -252,7 +252,7 @@ struct ConvertBatchMmt4DtoMmt4DPattern
   }
 };
 
-struct Convert3DPackto2DPackPattern : public OpRewritePattern<linalg::PackOp> {
+struct Convert3DPackto2DPackPattern : OpRewritePattern<linalg::PackOp> {
   using Base::Base;
 
   LogicalResult matchAndRewrite(linalg::PackOp packOp,
@@ -326,8 +326,7 @@ struct Convert3DPackto2DPackPattern : public OpRewritePattern<linalg::PackOp> {
   }
 };
 
-struct Convert5DUnPackto4DUnPackPattern
-    : public OpRewritePattern<linalg::UnPackOp> {
+struct Convert5DUnPackto4DUnPackPattern : OpRewritePattern<linalg::UnPackOp> {
   using Base::Base;
 
   LogicalResult matchAndRewrite(linalg::UnPackOp unpackOp,
@@ -409,7 +408,7 @@ struct Convert5DUnPackto4DUnPackPattern
 };
 
 struct CPUPrepareUkernelsPass
-    : public impl::CPUPrepareUkernelsPassBase<CPUPrepareUkernelsPass> {
+    : impl::CPUPrepareUkernelsPassBase<CPUPrepareUkernelsPass> {
   void getDependentDialects(DialectRegistry &registry) const override {
     registry.insert<linalg::LinalgDialect, arith::ArithDialect,
                     tensor::TensorDialect, scf::SCFDialect>();

compiler/src/iree/compiler/Codegen/Common/CPU/CPUPropagateDataLayout.cpp

@@ -33,7 +33,7 @@ namespace {
 /// introducing tensor.expand_shape op on the destination tensor. However, it is
 /// not common in practice, so it is not supported now.
 struct SinkDownCollapsingUnitDimsAcrossUnpack final
-    : public OpRewritePattern<linalg::UnPackOp> {
+    : OpRewritePattern<linalg::UnPackOp> {
   using Base::Base;
   LogicalResult matchAndRewrite(linalg::UnPackOp op,
                                 PatternRewriter &rewriter) const override {
@@ -147,7 +147,7 @@ struct SinkDownCollapsingUnitDimsAcrossUnpack final
 };
 
 struct CPUPropagateDataLayoutPass final
-    : public impl::CPUPropagateDataLayoutPassBase<CPUPropagateDataLayoutPass> {
+    : impl::CPUPropagateDataLayoutPassBase<CPUPropagateDataLayoutPass> {
   void getDependentDialects(DialectRegistry &registry) const override {
     registry.insert<linalg::LinalgDialect, tensor::TensorDialect>();
   }

compiler/src/iree/compiler/Codegen/Common/CombineLayoutTransformation.cpp

@@ -656,7 +656,7 @@ shouldDoReshapesByExpansion(IREE::Codegen::RelayoutCombinationScope scope) {
 /// the leaves of the chain. The leaves are simply relayout ops that have non
 /// relayout op users. The `controlFn` is a callback on the leaf OpResult that
 /// provides control over whether or not to insert a map_store op.
-struct InsertMapStoreOpPattern : public RewritePattern {
+struct InsertMapStoreOpPattern : RewritePattern {
   InsertMapStoreOpPattern(MLIRContext *context,
                           CombineRelayoutOpsControlFnRef controlFn = nullptr,
                           PatternBenefit benefit = 1)
@@ -1193,7 +1193,7 @@ FailureOr<MapLoadOp> foldIntoMapLoad(RewriterBase &rewriter, Operation *op,
 
 /// Pattern to fold consumer relayout ops into a producer map_load.
 struct FoldConsumerRelayoutIntoMapLoadPattern
-    : public OpRewritePattern<IREE::LinalgExt::MapLoadOp> {
+    : OpRewritePattern<IREE::LinalgExt::MapLoadOp> {
   using OpRewritePattern::OpRewritePattern;
 
   LogicalResult matchAndRewrite(IREE::LinalgExt::MapLoadOp mapLoadOp,
@@ -1249,7 +1249,7 @@ insertIdentityMapLoad(RewriterBase &rewriter, OpResult root,
 /// op consumers. The identity map_load can then be used to fold consumer
 /// relayout ops into it iteratively.
 struct InsertMapLoadOpPattern
-    : public OpRewritePattern<IREE::Codegen::LoadFromBufferOp> {
+    : OpRewritePattern<IREE::Codegen::LoadFromBufferOp> {
   InsertMapLoadOpPattern(MLIRContext *context, bool combineNonComplexChains)
       : OpRewritePattern(context),
         combineNonComplexChains(combineNonComplexChains) {}

compiler/src/iree/compiler/Codegen/Common/ConcretizePadResultShape.cpp

@@ -47,7 +47,7 @@ namespace {
 
 /// Concretizes tensor.pad op's result shape if its source op implements
 /// OffsetSizeAndStrideOpInterface. For example, pad(extract_slice).
-struct ConcretizePadResultShape final : public OpRewritePattern<tensor::PadOp> {
+struct ConcretizePadResultShape final : OpRewritePattern<tensor::PadOp> {
   using Base::Base;
 
   LogicalResult matchAndRewrite(tensor::PadOp padOp,

compiler/src/iree/compiler/Codegen/Common/ConvertBf16ArithToF32.cpp

@@ -63,7 +63,7 @@ Value convertRankedFloat(OpBuilder &builder, Type type, ValueRange inputs,
 
 // Converts from |SourceType| to |TargetType|.
 template <typename SourceType, typename TargetType>
-struct PrimitiveTypeConverter : public TypeConverter {
+struct PrimitiveTypeConverter : TypeConverter {
   explicit PrimitiveTypeConverter() {
     addConversion([](Type type) { return type; });
     addConversion([&](SourceType type) -> Type {
@@ -102,8 +102,7 @@ struct PrimitiveTypeConverter : public TypeConverter {
 };
 
 template <typename SourceType, typename TargetType>
-struct FloatTypeConverter
-    : public PrimitiveTypeConverter<SourceType, TargetType> {
+struct FloatTypeConverter : PrimitiveTypeConverter<SourceType, TargetType> {
   explicit FloatTypeConverter() {
     this->addSourceMaterialization(convertRankedFloat);
     this->addTargetMaterialization(convertRankedFloat);
@@ -112,7 +111,7 @@ struct FloatTypeConverter
 
 // Tries to completely convert a generic Operation.
 // This will process attributes, result types, and nested regions.
-struct GenericTypeConversionPattern : public ConversionPattern {
+struct GenericTypeConversionPattern : ConversionPattern {
   GenericTypeConversionPattern(MLIRContext *context,
                                TypeConverter &typeConverter)
       : ConversionPattern(typeConverter, MatchAnyOpTypeTag(), 0, context) {}
@@ -163,7 +162,7 @@ struct GenericTypeConversionPattern : public ConversionPattern {
 // unsigned integer values.
 template <typename OpTy, typename TypeTy,
           typename OperandToResultWidthLegalityRelation>
-struct ConvertTypeSensitiveArithCastOp : public OpConversionPattern<OpTy> {
+struct ConvertTypeSensitiveArithCastOp : OpConversionPattern<OpTy> {
   using OpConversionPattern<OpTy>::OpConversionPattern;
   LogicalResult
   matchAndRewrite(OpTy op, typename OpTy::Adaptor adaptor,
@@ -231,7 +230,7 @@ class PropagateCastF : public OpRewritePattern<SrcOp> {
 
 // Converts BF16s to F32s.
 struct PromoteBF16ToF32Converter
-    : public FloatTypeConverter<BFloat16Type, Float32Type> {
+    : FloatTypeConverter<BFloat16Type, Float32Type> {
   Type getTargetType(BFloat16Type type) override {
     return Float32Type::get(type.getContext());
   }

compiler/src/iree/compiler/Codegen/Common/ConvertToDestinationPassingStylePass.cpp

@@ -512,8 +512,7 @@ replaceUnpackEmptyWithAllocTensor(OpBuilder &b,
 }
 
 namespace {
-struct RemoveCstOutsDependency
-    : public OpInterfaceRewritePattern<linalg::LinalgOp> {
+struct RemoveCstOutsDependency : OpInterfaceRewritePattern<linalg::LinalgOp> {
   using OpInterfaceRewritePattern<linalg::LinalgOp>::OpInterfaceRewritePattern;
 
   LogicalResult matchAndRewrite(linalg::LinalgOp op,
@@ -577,7 +576,7 @@ struct RemoveCstOutsDependency
 /// ```
 /// This is a workaround for #11273 while a proper fix lands.
 struct SwitchStoreOfIfResultValue
-    : public OpRewritePattern<IREE::TensorExt::DispatchTensorStoreOp> {
+    : OpRewritePattern<IREE::TensorExt::DispatchTensorStoreOp> {
   using Base::Base;
 
   LogicalResult matchAndRewrite(IREE::TensorExt::DispatchTensorStoreOp storeOp,

compiler/src/iree/compiler/Codegen/Common/ConvertUnsupportedFloatArithPass.cpp

@@ -756,7 +756,7 @@ struct ExtFFromSmallFloat final : OpRewritePattern<arith::ExtFOp> {
 };
 
 struct ConvertUnsupportedFloatArithPass final
-    : public impl::ConvertUnsupportedFloatArithPassBase<
+    : impl::ConvertUnsupportedFloatArithPassBase<
           ConvertUnsupportedFloatArithPass> {
   void runOnOperation() override;
   using Base::Base;

compiler/src/iree/compiler/Codegen/Common/ConvertUnsupportedFloatToIntBuffers.cpp

@@ -160,7 +160,7 @@ struct ConvertMemRefAlloc final : OpConversionPattern<memref::AllocOp> {
 
 // Tries to completely convert a generic Operation.
 // This will process attributes, result types, and nested regions.
-struct GenericTypeConversionPattern : public ConversionPattern {
+struct GenericTypeConversionPattern : ConversionPattern {
   GenericTypeConversionPattern(TypeConverter &typeConverter,
                                MLIRContext *context)
       : ConversionPattern(typeConverter, MatchAnyOpTypeTag(), 0, context) {}

compiler/src/iree/compiler/Codegen/Common/ConvertWorkgroupForallToPCF.cpp

@@ -26,7 +26,7 @@ struct ConvertWorkgroupForall : OpRewritePattern<scf::ForallOp> {
 };
 
 struct ConvertWorkgroupForallToPCFPass final
-    : public impl::ConvertWorkgroupForallToPCFPassBase<
+    : impl::ConvertWorkgroupForallToPCFPassBase<
           ConvertWorkgroupForallToPCFPass> {
   void runOnOperation() override;
   using Base::Base;