Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Vulkan] Some fixes of Vulkan codegen #16405

Closed
wants to merge 1 commit into from
Closed

[Vulkan] Some fixes of Vulkan codegen #16405

wants to merge 1 commit into from

Conversation

spectrometerHBH
Copy link
Contributor

  • add exp2 dispatch
  • handle thread invariant op
  • disable smem merge for vulkan

- add exp2 dispatch
- handle thread invariant op
- disable smem merge for vulkan
@junrushao
Copy link
Member

Cherry-picked to mlc-ai/relax#309

junrushao pushed a commit to junrushao/tvm that referenced this pull request Jan 17, 2024
This PR upstreams a few commits that recovers the unity branch from
broken wheel packages. It includes the following changes:

- Fix MSVC build in `pipe.h` where `DWORD` is not cast to proper return
  type (mlc-ai/relax#306);
- Fix MSVC build warnings on not recognizing "#pragma GCC"
  (mlc-ai/relax#307);
- Fix NVCC build warnings where it fails to infer if "[[noreturn]]"
  actually does not return (mlc-ai/relax#308);
- Fix ROCM/Vulkan backend which fails compilation for operators like group
  GEMM, paged attention, etc. (apache#16404,
  apache#16405)
junrushao added a commit to junrushao/tvm that referenced this pull request Jan 17, 2024
This PR upstreams a few commits that recovers the unity branch from
broken wheel packages. It includes the following changes:

- Fix MSVC build in `pipe.h` where `DWORD` is not cast to proper return
  type (mlc-ai/relax#306);
- Fix MSVC build warnings on not recognizing "#pragma GCC"
  (mlc-ai/relax#307);
- Fix NVCC build warnings where it fails to infer if "[[noreturn]]"
  actually does not return (mlc-ai/relax#308);
- Fix ROCM/Vulkan backend which fails compilation for operators like group
  GEMM, paged attention, etc. (apache#16404,
  apache#16405)

Co-authored-by: Bohan Hou <[email protected]>
Co-authored-by: Lesheng Jin <[email protected]>
junrushao added a commit to junrushao/tvm that referenced this pull request Jan 17, 2024
This PR upstreams a few commits that recovers the unity branch from
broken wheel packages. It includes the following changes:

- Fix MSVC build in `pipe.h` where `DWORD` is not cast to proper return
  type (mlc-ai/relax#306);
- Fix MSVC build warnings on not recognizing "#pragma GCC"
  (mlc-ai/relax#307);
- Fix NVCC build warnings where it fails to infer if "[[noreturn]]"
  actually does not return (mlc-ai/relax#308);
- Fix ROCM/Vulkan backend which fails compilation for operators like group
  GEMM, paged attention, etc. (apache#16404,
  apache#16405)

Co-authored-by: Bohan Hou <[email protected]>
Co-authored-by: Lesheng Jin <[email protected]>
junrushao added a commit to mlc-ai/relax that referenced this pull request Jan 17, 2024
This PR upstreams a few commits that recovers the unity branch from
broken wheel packages. It includes the following changes:

- Fix MSVC build in `pipe.h` where `DWORD` is not cast to proper return
  type (#306);
- Fix MSVC build warnings on not recognizing "#pragma GCC"
  (#307);
- Fix NVCC build warnings where it fails to infer if "[[noreturn]]"
  actually does not return (#308);
- Fix ROCM/Vulkan backend which fails compilation for operators like group
  GEMM, paged attention, etc. (apache/tvm#16404,
  apache/tvm#16405)

Co-authored-by: Bohan Hou <[email protected]>
Co-authored-by: Lesheng Jin <[email protected]>
junrushao added a commit to mlc-ai/relax that referenced this pull request Jan 17, 2024
This PR upstreams a few commits that recovers the unity branch from
broken wheel packages. It includes the following changes:

- Fix MSVC build in `pipe.h` where `DWORD` is not cast to proper return
  type (#306);
- Fix MSVC build warnings on not recognizing "#pragma GCC"
  (#307);
- Fix NVCC build warnings where it fails to infer if "[[noreturn]]"
  actually does not return (#308);
- Fix ROCM/Vulkan backend which fails compilation for operators like group
  GEMM, paged attention, etc. (apache/tvm#16404,
  apache/tvm#16405)

Co-authored-by: Bohan Hou <[email protected]>
Co-authored-by: Lesheng Jin <[email protected]>
tqchen pushed a commit that referenced this pull request Jan 17, 2024
This PR upstreams a few commits that recovers the unity branch from
broken wheel packages. It includes the following changes:

- Fix MSVC build in `pipe.h` where `DWORD` is not cast to proper return
  type (mlc-ai/relax#306);
- Fix MSVC build warnings on not recognizing "#pragma GCC"
  (mlc-ai/relax#307);
- Fix NVCC build warnings where it fails to infer if "[[noreturn]]"
  actually does not return (mlc-ai/relax#308);
- Fix ROCM/Vulkan backend which fails compilation for operators like group
  GEMM, paged attention, etc. (#16404,
  #16405)

Co-authored-by: Bohan Hou <[email protected]>
Co-authored-by: Lesheng Jin <[email protected]>
@tqchen
Copy link
Member

tqchen commented Jan 17, 2024

@tvm-bot rerun

junrushao added a commit to mlc-ai/relax that referenced this pull request Jan 21, 2024
This PR upstreams a few commits that recovers the unity branch from
broken wheel packages. It includes the following changes:

- Fix MSVC build in `pipe.h` where `DWORD` is not cast to proper return
  type (#306);
- Fix MSVC build warnings on not recognizing "#pragma GCC"
  (#307);
- Fix NVCC build warnings where it fails to infer if "[[noreturn]]"
  actually does not return (#308);
- Fix ROCM/Vulkan backend which fails compilation for operators like group
  GEMM, paged attention, etc. (apache/tvm#16404,
  apache/tvm#16405)

Co-authored-by: Bohan Hou <[email protected]>
Co-authored-by: Lesheng Jin <[email protected]>
@Hzfengsy
Copy link
Member

Please rebase :)

@junrushao
Copy link
Member

I think it's part of #16414 that has been merged

@tqchen tqchen closed this Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants