https://github.com/gysit approved this pull request. LGTM If possible we should use DenseI32ArrayAttr. It is stores the elements using an actual vector of integers which is more efficient and less casting should be necessary. https://github.com/llvm/llvm-project/pull/141299