[Mlir-commits] [mlir] [MLIR] support dynamic indexing in `VectorEmulateNarrowTypes` (PR #114169)

[Han-Chung Wang]

================
@@ -380,25 +433,27 @@ struct ConvertVectorLoad final : OpConversionPattern<vector::LoadOp> {
             ? getConstantIntValue(linearizedInfo.intraDataOffset)
             : 0;
 
-    if (!foldedIntraVectorOffset) {
-      // unimplemented case for dynamic intra vector offset
-      return failure();
-    }
-
+    // always load enough elements which can cover the original elements
----------------
hanhanW wrote:

style nit: complete the sentence in the comment.

```suggestion
    // Always load enough elements which can cover the original elements.
```

https://google.github.io/styleguide/cppguide.html#Punctuation,_Spelling_and_Grammar

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