[Mlir-commits] [mlir] 37cf1fc - [mlir][vector] Fix missed `return` in ExtractStridedSliceOp::fold (#165669)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Fri Oct 31 01:43:29 PDT 2025


Author: Haocong Lu
Date: 2025-10-31T16:43:25+08:00
New Revision: 37cf1fc62c8d95bc2b7487ef1302dc472606d66e

URL: https://github.com/llvm/llvm-project/commit/37cf1fc62c8d95bc2b7487ef1302dc472606d66e
DIFF: https://github.com/llvm/llvm-project/commit/37cf1fc62c8d95bc2b7487ef1302dc472606d66e.diff

LOG: [mlir][vector] Fix missed `return` in ExtractStridedSliceOp::fold (#165669)

Fix missed `return` when folding splat ConstantOp, it could work well
probably because of good compatibility of
`foldExtractStridedSliceNonSplatConstant`.

Added: 
    

Modified: 
    mlir/lib/Dialect/Vector/IR/VectorOps.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Dialect/Vector/IR/VectorOps.cpp b/mlir/lib/Dialect/Vector/IR/VectorOps.cpp
index ad8255a95cb4e..ae3423c40040d 100644
--- a/mlir/lib/Dialect/Vector/IR/VectorOps.cpp
+++ b/mlir/lib/Dialect/Vector/IR/VectorOps.cpp
@@ -4336,7 +4336,7 @@ OpFoldResult ExtractStridedSliceOp::fold(FoldAdaptor adaptor) {
   // ExtractStridedSliceOp(splat ConstantOp) -> ConstantOp.
   if (auto splat =
           llvm::dyn_cast_if_present<SplatElementsAttr>(adaptor.getSource()))
-    DenseElementsAttr::get(getType(), splat.getSplatValue<Attribute>());
+    return DenseElementsAttr::get(getType(), splat.getSplatValue<Attribute>());
 
   // ExtractStridedSliceOp(non-splat ConstantOp) -> ConstantOp.
   return foldExtractStridedSliceNonSplatConstant(*this, adaptor.getSource());


        


More information about the Mlir-commits mailing list