Merge pull request #1792 from JayFoxRox/avoid-uninitialised
Avoid uninitialised in hw renderer + Only sync depth if necessary
This commit is contained in:
		| @@ -104,7 +104,6 @@ RasterizerOpenGL::RasterizerOpenGL() : shader_dirty(true) { | ||||
|  | ||||
|     // Sync fixed function OpenGL state | ||||
|     SyncCullMode(); | ||||
|     SyncDepthModifiers(); | ||||
|     SyncBlendEnabled(); | ||||
|     SyncBlendFuncs(); | ||||
|     SyncBlendColor(); | ||||
| @@ -259,8 +258,10 @@ void RasterizerOpenGL::NotifyPicaRegisterChanged(u32 id) { | ||||
|  | ||||
|     // Depth modifiers | ||||
|     case PICA_REG_INDEX(viewport_depth_range): | ||||
|         SyncDepthScale(); | ||||
|         break; | ||||
|     case PICA_REG_INDEX(viewport_depth_near_plane): | ||||
|         SyncDepthModifiers(); | ||||
|         SyncDepthOffset(); | ||||
|         break; | ||||
|  | ||||
|     // Depth buffering | ||||
| @@ -880,6 +881,8 @@ void RasterizerOpenGL::SetShader() { | ||||
|         glUniformBlockBinding(current_shader->shader.handle, block_index, 0); | ||||
|  | ||||
|         // Update uniforms | ||||
|         SyncDepthScale(); | ||||
|         SyncDepthOffset(); | ||||
|         SyncAlphaTest(); | ||||
|         SyncCombinerColor(); | ||||
|         auto& tev_stages = Pica::g_state.regs.GetTevStages(); | ||||
| @@ -922,13 +925,20 @@ void RasterizerOpenGL::SyncCullMode() { | ||||
|     } | ||||
| } | ||||
|  | ||||
| void RasterizerOpenGL::SyncDepthModifiers() { | ||||
| void RasterizerOpenGL::SyncDepthScale() { | ||||
|     float depth_scale = Pica::float24::FromRaw(Pica::g_state.regs.viewport_depth_range).ToFloat32(); | ||||
|     float depth_offset = Pica::float24::FromRaw(Pica::g_state.regs.viewport_depth_near_plane).ToFloat32(); | ||||
|     if (depth_scale != uniform_block_data.data.depth_scale) { | ||||
|         uniform_block_data.data.depth_scale = depth_scale; | ||||
|         uniform_block_data.dirty = true; | ||||
|     } | ||||
| } | ||||
|  | ||||
|     uniform_block_data.data.depth_scale = depth_scale; | ||||
|     uniform_block_data.data.depth_offset = depth_offset; | ||||
|     uniform_block_data.dirty = true; | ||||
| void RasterizerOpenGL::SyncDepthOffset() { | ||||
|     float depth_offset = Pica::float24::FromRaw(Pica::g_state.regs.viewport_depth_near_plane).ToFloat32(); | ||||
|     if (depth_offset != uniform_block_data.data.depth_offset) { | ||||
|         uniform_block_data.data.depth_offset = depth_offset; | ||||
|         uniform_block_data.dirty = true; | ||||
|     } | ||||
| } | ||||
|  | ||||
| void RasterizerOpenGL::SyncBlendEnabled() { | ||||
|   | ||||
| @@ -339,8 +339,11 @@ private: | ||||
|     /// Syncs the cull mode to match the PICA register | ||||
|     void SyncCullMode(); | ||||
|  | ||||
|     /// Syncs the depth scale and offset to match the PICA registers | ||||
|     void SyncDepthModifiers(); | ||||
|     /// Syncs the depth scale to match the PICA register | ||||
|     void SyncDepthScale(); | ||||
|  | ||||
|     /// Syncs the depth offset to match the PICA register | ||||
|     void SyncDepthOffset(); | ||||
|  | ||||
|     /// Syncs the blend enabled status to match the PICA register | ||||
|     void SyncBlendEnabled(); | ||||
| @@ -413,7 +416,7 @@ private: | ||||
|         UniformData data; | ||||
|         bool lut_dirty[6]; | ||||
|         bool dirty; | ||||
|     } uniform_block_data; | ||||
|     } uniform_block_data = {}; | ||||
|  | ||||
|     std::array<SamplerInfo, 3> texture_samplers; | ||||
|     OGLVertexArray vertex_array; | ||||
| @@ -422,5 +425,5 @@ private: | ||||
|     OGLFramebuffer framebuffer; | ||||
|  | ||||
|     std::array<OGLTexture, 6> lighting_luts; | ||||
|     std::array<std::array<GLvec4, 256>, 6> lighting_lut_data; | ||||
|     std::array<std::array<GLvec4, 256>, 6> lighting_lut_data{}; | ||||
| }; | ||||
|   | ||||
		Reference in New Issue
	
	Block a user
	 bunnei
					bunnei