From 4eb1f1fab35d0f386b458bf1da4396bbeb00b04f Mon Sep 17 00:00:00 2001 From: Adrian Kuegel Date: Wed, 8 Mar 2023 10:39:21 +0100 Subject: [PATCH] [mlir] Apply ClangTidyLegacy findings (NFC) - 'override' is redundant since the function is already declared 'final'. - 'virtual' is redundant since the function is already declared 'override'. --- mlir/lib/Conversion/VectorToLLVM/ConvertVectorToLLVM.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mlir/lib/Conversion/VectorToLLVM/ConvertVectorToLLVM.cpp b/mlir/lib/Conversion/VectorToLLVM/ConvertVectorToLLVM.cpp index f705284..c56d03f 100644 --- a/mlir/lib/Conversion/VectorToLLVM/ConvertVectorToLLVM.cpp +++ b/mlir/lib/Conversion/VectorToLLVM/ConvertVectorToLLVM.cpp @@ -819,7 +819,7 @@ public: LogicalResult matchAndRewrite(vector::MaskOp maskOp, OpAdaptor adaptor, - ConversionPatternRewriter &rewriter) const override final { + ConversionPatternRewriter &rewriter) const final { // Match against the maskable operation kind. auto maskedOp = llvm::dyn_cast_or_null(maskOp.getMaskableOp()); if (!maskedOp) @@ -841,7 +841,7 @@ public: using VectorMaskOpConversionBase< vector::ReductionOp>::VectorMaskOpConversionBase; - virtual LogicalResult matchAndRewriteMaskableOp( + LogicalResult matchAndRewriteMaskableOp( vector::MaskOp maskOp, MaskableOpInterface maskableOp, ConversionPatternRewriter &rewriter) const override { auto reductionOp = cast(maskableOp.getOperation()); -- 2.7.4