Rationale:
When lowering to LLVM for different rank insert (n vs k), the offset
arrays needs to drop one dimension (becomes n-1), but the strides
array needs to be preserved (remains k). With regression test.
Note that this example was actually in the documentation, so
extra important to do it right :-)
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Unit tests: fail. 62352 tests passed, 1 failed and 839 were skipped.
failed: libc++.std/containers/sequences/array/array_creation/to_array.fail.cpp
clang-tidy: pass.
clang-format: pass.
Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml
Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.
Unit tests: fail. 62352 tests passed, 1 failed and 839 were skipped.
failed: libc++.std/containers/sequences/array/array_creation/to_array.fail.cpp
clang-tidy: pass.
clang-format: pass.
Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml
Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.
Unit tests: fail. 62352 tests passed, 1 failed and 839 were skipped.
failed: libc++.std/containers/sequences/array/array_creation/to_array.fail.cpp
clang-tidy: pass.
clang-format: pass.
Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml
Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.
Unit tests: fail. 62352 tests passed, 1 failed and 839 were skipped.
failed: libc++.std/containers/sequences/array/array_creation/to_array.fail.cpp
clang-tidy: pass.
clang-format: pass.
Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml
Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.
Unit tests: fail. 62343 tests passed, 1 failed and 839 were skipped.
failed: libc++.std/containers/sequences/array/array_creation/to_array.fail.cpp
clang-tidy: pass.
clang-format: pass.
Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml
Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.
LGTM after changing the test to capture %arg0 and %arg1. Thanks!
mlir/test/Conversion/VectorToLLVM/vector-to-llvm.mlir | ||
---|---|---|
7 | Nit: these changes to the test look orthogonal to the commit | |
551 | Let's capture SSA names for arguments as well: CHECK-LABEL: llvm.func @insert_strided_slice3 CHECK-SAME: %[[arg0:.*]]: !llvm< CHECK-SAME: %[[arg1:.*]]: !llvm< (CHECK-LABEL does not support capture) |
Unit tests: fail. 62343 tests passed, 1 failed and 839 were skipped.
failed: libc++.std/containers/sequences/array/array_creation/to_array.fail.cpp
clang-tidy: pass.
clang-format: pass.
Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml
Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.
Nit: these changes to the test look orthogonal to the commit