[Mlir-commits] [mlir] [MLIR] Fix VectorEmulateNarrowType constant op mask bug (PR #116064)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Thu Nov 14 13:41:36 PST 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff ec066d30e29fce388b1722971970d73ec65f14fb dbb73f6b7b13359fad84f93e10fbe4e758e4809f --extensions cpp -- mlir/lib/Dialect/Vector/Transforms/VectorEmulateNarrowType.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/mlir/lib/Dialect/Vector/Transforms/VectorEmulateNarrowType.cpp b/mlir/lib/Dialect/Vector/Transforms/VectorEmulateNarrowType.cpp
index 8a7b93b78f..1c1e509f3b 100644
--- a/mlir/lib/Dialect/Vector/Transforms/VectorEmulateNarrowType.cpp
+++ b/mlir/lib/Dialect/Vector/Transforms/VectorEmulateNarrowType.cpp
@@ -171,7 +171,7 @@ static FailureOr<Operation *> getCompressedMaskOp(OpBuilder &rewriter,
             // coverage size is mulitiple of bytes). The new mask will be like
             // this before compressing:
             //
-            // %new_mask = 
+            // %new_mask =
             //   [false, false, true, false, true, false, false, false]
             auto denseAttr =
                 dyn_cast<DenseIntElementsAttr>(constantOp.getValue());
@@ -182,7 +182,8 @@ static FailureOr<Operation *> getCompressedMaskOp(OpBuilder &rewriter,
                               denseAttr.template value_end<bool>());
             maskValues.resize(numElements * numSrcElemsPerDest, false);
 
-            // Compressing by combining every block of `numSrcElemsPerDest` elements togehter:
+            // Compressing by combining every block of `numSrcElemsPerDest`
+            // elements togehter:
             SmallVector<bool> compressedMaskValues;
             for (size_t i = 0; i < maskValues.size(); i += numSrcElemsPerDest) {
               bool combinedValue = false;

``````````

</details>


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


More information about the Mlir-commits mailing list