Skip to content

[SPIRV] Allow globallycoherent on DescriptorHeap accesses (#7740)#8513

Open
SteveUrquhart wants to merge 1 commit into
microsoft:mainfrom
SteveUrquhart:allow-globallycoherent-spirv
Open

[SPIRV] Allow globallycoherent on DescriptorHeap accesses (#7740)#8513
SteveUrquhart wants to merge 1 commit into
microsoft:mainfrom
SteveUrquhart:allow-globallycoherent-spirv

Conversation

@SteveUrquhart

Copy link
Copy Markdown
Contributor

This PR allows the SpirvEmitter to emit globallycoherent in combination with ResourceDescriptorHeap.

@SteveUrquhart

Copy link
Copy Markdown
Contributor Author

This addresses #7740

@llvm-beanz

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Comment thread tools/clang/lib/SPIRV/SpirvEmitter.cpp Outdated
Comment thread tools/clang/lib/SPIRV/SpirvEmitter.cpp Outdated
@SteveUrquhart
SteveUrquhart force-pushed the allow-globallycoherent-spirv branch 2 times, most recently from 60547e1 to b053280 Compare June 30, 2026 15:56
@SteveUrquhart
SteveUrquhart force-pushed the allow-globallycoherent-spirv branch from b053280 to da1dd30 Compare June 30, 2026 16:03
@SteveUrquhart

Copy link
Copy Markdown
Contributor Author

@llvm-beanz, Thanks for showing me and implementing the AST fix. This PR became trivial.

@dnovillo dnovillo added the spirv Work related to SPIR-V label Jul 13, 2026
@llvm-beanz

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@SteveUrquhart

Copy link
Copy Markdown
Contributor Author

Hi @llvm-beanz , can we merge this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

spirv Work related to SPIR-V

Projects

Status: New

Development

Successfully merging this pull request may close these issues.

3 participants