[Mlir-commits] [mlir] [mlir][vector]advance support extract insert under dynamic case. (PR #121631)

Oleksandr Alex Zinenko llvmlistbot at llvm.org
Mon Jan 6 06:56:48 PST 2025


================
@@ -1231,6 +1251,25 @@ class VectorInsertOpConversion
 
     SmallVector<OpFoldResult> positionVec = getMixedValues(
         adaptor.getStaticPosition(), adaptor.getDynamicPosition(), rewriter);
+    for (unsigned idx = 0; idx < positionVec.size(); ++idx) {
----------------
ftynse wrote:

It is a bad practice to copy large chunks of code. Use a function instead.

https://github.com/llvm/llvm-project/pull/121631


More information about the Mlir-commits mailing list