[all-commits] [llvm/llvm-project] b301a9: [mlir][spirv] Add integration tests for `vector.in...

Angel Zhang via All-commits all-commits at lists.llvm.org
Mon Jun 3 07:13:01 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b301a98b33d75813d73838c1bd4c47024d044af6
      https://github.com/llvm/llvm-project/commit/b301a98b33d75813d73838c1bd4c47024d044af6
  Author: Angel Zhang <angel.zhang at amd.com>
  Date:   2024-06-03 (Mon, 03 Jun 2024)

  Changed paths:
    M mlir/lib/Conversion/GPUToSPIRV/CMakeLists.txt
    M mlir/lib/Conversion/GPUToSPIRV/GPUToSPIRVPass.cpp
    A mlir/test/mlir-vulkan-runner/vector-interleave.mlir
    A mlir/test/mlir-vulkan-runner/vector-shuffle.mlir

  Log Message:
  -----------
  [mlir][spirv] Add integration tests for `vector.interleave` and `vector.shuffle` (#93858)

This PR tries to reland #93595 which was reverted in #93732 due to some
issues. The original PR:
- Add integration test for  `vector.shuffle` and `vector.interleave`
- Add `VectorToSPIRV` patterns to `GPUToSPIRVPass`

Description of the issue:
-
https://github.com/llvm/llvm-project/pull/93595#issuecomment-2138541700
- Using either `vector.load` or `vector.store` in the kernel function
will cause the validation layer to report an error
- Trying to bypass the issue by using `memref.load` and `memref.store`
to load/store individual elements from/to the vectors, and populate the
vectors using `vector.insertelement` and `vector.extractelement`
instead.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list