Skip to content

fix: #2214 Vulkan uses gl_InstanceIndex and gl_VertexIndex instead of gl_InstanceId and gl_VertexId as the latter are not supported. #2217

Merged
Eideren merged 1 commit into
stride3d:masterfrom
johang88:2214-VulkanFixes
Apr 21, 2024
Merged

Conversation

@johang88

@johang88 johang88 commented Apr 5, 2024

Copy link
Copy Markdown
Contributor

PR Details

Vulkan uses gl_InstanceIndex and gl_VertexIndex instead of gl_InstanceId and gl_VertexId as the latter are not supported.

Related Issue

#2214

Motivation and Context

gl_InstanceId and gl_VertexId are removed in https://github.com/KhronosGroup/GLSL/blob/main/extensions/khr/GL_KHR_vulkan_glsl.txt

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My change requires a change to the documentation.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have built and run the editor to try this change out.

@Eideren

Eideren commented Apr 21, 2024

Copy link
Copy Markdown
Collaborator

I'm guessing those are strictly nomenclature changes then ? Thanks !

@Eideren Eideren merged commit d7a47a2 into stride3d:master Apr 21, 2024
@Eideren Eideren changed the title #2214 Vulkan uses gl_InstanceIndex and gl_VertexIndex instead of gl_InstanceId and gl_VertexId as the latter are not supported. fix: #2214 Vulkan uses gl_InstanceIndex and gl_VertexIndex instead of gl_InstanceId and gl_VertexId as the latter are not supported. Apr 21, 2024
phr00t pushed a commit to phr00t/FocusEngine that referenced this pull request Apr 29, 2024
… of gl_InstanceId and gl_VertexId (stride3d#2217)

# Conflicts:
#	sources/shaders/Xenko.Core.Shaders/Convertor/HlslToGlslConvertor.cs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants