NaxdyOrg/Ryujinx
1
0
Fork 0
forked from Mirror/Ryujinx
Code Pull requests Activity

Change disabled vertex attribute value to (0, 0, 0, 1) (#2573)

Browse source 
This seems to be the default value when the vertex attribute is disabled, or components aren't defined.

This fixes a regression from #2307 in SMO where a plant in the Wooded Kingdom would draw slightly differently in the depth prepass, leading to depth test failing later on.

GDK has stated that the specific case in Gundam only expects x and y to be 0, and Vulkan's undefined value for z does appear to be 0 when a vertex attribute type does not have that component, hence the value (0, 0, 0, 1).

This worked in Vulkan despite also providing an all 0s attribute due to the vertex attribute binding being R32Float, so the other values were undefined. It should be changed there separately.
This commit is contained in:
riperiperi 2021-08-20 22:09:30 +01:00 committed by GitHub
parent bdc1f91a5b
commit 6c76bc3bc0
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
Ryujinx.Graphics.OpenGL/VertexArray.cs
View file

@ -176,7 +176,7 @@ namespace Ryujinx.Graphics.OpenGL
{ {
_vertexAttribsInUse &= ~mask; _vertexAttribsInUse &= ~mask;
GL.DisableVertexAttribArray(index); GL.DisableVertexAttribArray(index);
GL.VertexAttrib4(index, 0f, 0f, 0f, 0f); GL.VertexAttrib4(index, 0f, 0f, 0f, 1f);
} }
} }