Skip to content

Implement extension GL_EXT_descriptor_heap.#4155

Merged
jeremy-lunarg merged 1 commit intoKhronosGroup:mainfrom
ShchchowAMD:EXT_descriptor_heap
Jan 23, 2026
Merged

Implement extension GL_EXT_descriptor_heap.#4155
jeremy-lunarg merged 1 commit intoKhronosGroup:mainfrom
ShchchowAMD:EXT_descriptor_heap

Conversation

@ShchchowAMD
Copy link
Copy Markdown
Contributor

Support GL_EXT_descriptor_heap in glslang.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 21, 2026

CLA assistant check
All committers have signed the CLA.

@ShchchowAMD
Copy link
Copy Markdown
Contributor Author

@jeremy-lunarg
Copy link
Copy Markdown
Contributor

@ShchchowAMD I started cleaning up some warning/errors but when I got to the error, GlslangToSpv.cpp:6069:33: error: unused parameter ‘members’, I realized you may have some unfinished functionality. Please take a look.

@spencer-lunarg
Copy link
Copy Markdown
Contributor

We need to update the known_goods.json with

  • SPIRV-Header commit 04f10f650d514df88b76d25e83db360142c7b174
  • SPIRV-Tools commit c2cf37e46552dd71f5cfe3f53330853cf081c066

@ShchchowAMD
Copy link
Copy Markdown
Contributor Author

Hi, sorry for the late reply, I'll fix them today, thanks for reminding and all the help yesterday! Sorry for those issues not found before in my local.

@ShchchowAMD
Copy link
Copy Markdown
Contributor Author

Just removed unused variable and parameters in this PR. They are wrongly be kept during development, current implementation for decorating structure heap type is finished.

Copy link
Copy Markdown
Contributor

@jeremy-lunarg jeremy-lunarg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some questions but otherwise looks okay.

@ShchchowAMD
Copy link
Copy Markdown
Contributor Author

Rebased and clean up commented extension macro declaration.

@jeremy-lunarg
Copy link
Copy Markdown
Contributor

@ShchchowAMD Thanks for the rebase. I was in the middle of doing that as well, but appreciate you taking over. Please squash your commits into a single commit to help triage future bisects.

@ShchchowAMD
Copy link
Copy Markdown
Contributor Author

@jeremy-lunarg Hi, I squash those commits and ready to merge now. But I found I may don't have access to process merge operation here.

@jeremy-lunarg
Copy link
Copy Markdown
Contributor

Thanks for the fixes, rebase, and squash!

@jeremy-lunarg jeremy-lunarg merged commit c8d3e06 into KhronosGroup:main Jan 23, 2026
22 checks passed
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