[Mlir-commits] [mlir] [mlir][tosa] Improve slice op verifier (PR #181889)

[llvmlistbot at llvm.org]

github-actions[bot] wrote:

<!--PREMERGE ADVISOR COMMENT: Windows-->
# :window: Windows x64 Test Results

The build failed before running any tests. Click on a failure below to see the details.

<details>
<summary>[code=1] tools/mlir/lib/Dialect/Tosa/CMakeFiles/obj.MLIRTosaDialect.dir/IR/TosaOps.cpp.obj</summary>

```
FAILED: [code=1] tools/mlir/lib/Dialect/Tosa/CMakeFiles/obj.MLIRTosaDialect.dir/IR/TosaOps.cpp.obj
sccache C:\clang\clang-msvc\bin\clang-cl.exe  /nologo -TP -DUNICODE -D_CRT_NONSTDC_NO_DEPRECATE -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_DEPRECATE -D_CRT_SECURE_NO_WARNINGS -D_GLIBCXX_ASSERTIONS -D_HAS_EXCEPTIONS=0 -D_SCL_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -D_UNICODE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -Itools\mlir\lib\Dialect\Tosa -IC:\_work\llvm-project\llvm-project\mlir\lib\Dialect\Tosa -Itools\mlir\include -IC:\_work\llvm-project\llvm-project\mlir\include -Iinclude -IC:\_work\llvm-project\llvm-project\llvm\include /DWIN32 /D_WINDOWS   /Zc:inline /Zc:__cplusplus /Oi /Brepro /bigobj /permissive- -Werror=unguarded-availability-new /W4  -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -Wimplicit-fallthrough -Wcovered-switch-default -Wno-noexcept-type -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wsuggest-override -Wstring-conversion -Wno-pass-failed -Wmisleading-indentation -Wctad-maybe-unsupported /Gw -Wundef -Werror=mismatched-tags -Werror=global-constructors /O2 /Ob2  -MD -UNDEBUG /EHs-c- /GR- -std:c++17 /showIncludes /Fotools\mlir\lib\Dialect\Tosa\CMakeFiles\obj.MLIRTosaDialect.dir\IR\TosaOps.cpp.obj /Fdtools\mlir\lib\Dialect\Tosa\CMakeFiles\obj.MLIRTosaDialect.dir\ -c -- C:\_work\llvm-project\llvm-project\mlir\lib\Dialect\Tosa\IR\TosaOps.cpp
C:\_work\llvm-project\llvm-project\mlir\lib\Dialect\Tosa\IR\TosaOps.cpp(2317,23): error: decomposition declaration cannot be declared with type 'const int64_t' (aka 'const long long'); declared type must be 'auto' or reference to 'auto'
2317 |     for (const int64_t[start, size, inputDim] :
|                       ^
C:\_work\llvm-project\llvm-project\mlir\lib\Dialect\Tosa\IR\TosaOps.cpp(2317,23): error: no viable conversion from 'value_type' (aka 'std::tuple<long long &, long long &, long long &>') to 'const int64_t' (aka 'const long long')
2317 |     for (const int64_t[start, size, inputDim] :
|                       ^                       ~
C:\_work\llvm-project\llvm-project\llvm\include\llvm/ADT/STLExtras.h(804,12): note: selected 'begin' function with iterator type 'iterator' (aka 'zip_first<long long *, long long *, long long *>')
804 |   iterator begin() { return begin_impl(IndexSequence{}); }
|            ^
2 errors generated.
```
</details>

If these failures are unrelated to your changes (for example tests are broken or flaky at HEAD), please open an issue at https://github.com/llvm/llvm-project/issues and add the `infrastructure` label.

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