mirror of
				https://github.com/PabloMK7/citra.git
				synced 2025-10-30 21:30:04 +00:00 
			
		
		
		
	OpenGL: Use uniforms for dist_atten_bias and dist_atten_scale
This commit is contained in:
		
							parent
							
								
									e054e55ca2
								
							
						
					
					
						commit
						068bd6f728
					
				
					 3 changed files with 84 additions and 8 deletions
				
			
		|  | @ -518,6 +518,58 @@ void RasterizerOpenGL::NotifyPicaRegisterChanged(u32 id) { | |||
|         SyncLightPosition(7); | ||||
|         break; | ||||
| 
 | ||||
|     // Fragment lighting distance attenuation bias
 | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[0].dist_atten_bias, 0x014A + 0 * 0x10): | ||||
|         SyncLightDistanceAttenuationBias(0); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[1].dist_atten_bias, 0x014A + 1 * 0x10): | ||||
|         SyncLightDistanceAttenuationBias(1); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[2].dist_atten_bias, 0x014A + 2 * 0x10): | ||||
|         SyncLightDistanceAttenuationBias(2); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[3].dist_atten_bias, 0x014A + 3 * 0x10): | ||||
|         SyncLightDistanceAttenuationBias(3); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[4].dist_atten_bias, 0x014A + 4 * 0x10): | ||||
|         SyncLightDistanceAttenuationBias(4); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[5].dist_atten_bias, 0x014A + 5 * 0x10): | ||||
|         SyncLightDistanceAttenuationBias(5); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[6].dist_atten_bias, 0x014A + 6 * 0x10): | ||||
|         SyncLightDistanceAttenuationBias(6); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[7].dist_atten_bias, 0x014A + 7 * 0x10): | ||||
|         SyncLightDistanceAttenuationBias(7); | ||||
|         break; | ||||
| 
 | ||||
|     // Fragment lighting distance attenuation scale
 | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[0].dist_atten_scale, 0x014B + 0 * 0x10): | ||||
|         SyncLightDistanceAttenuationScale(0); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[1].dist_atten_scale, 0x014B + 1 * 0x10): | ||||
|         SyncLightDistanceAttenuationScale(1); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[2].dist_atten_scale, 0x014B + 2 * 0x10): | ||||
|         SyncLightDistanceAttenuationScale(2); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[3].dist_atten_scale, 0x014B + 3 * 0x10): | ||||
|         SyncLightDistanceAttenuationScale(3); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[4].dist_atten_scale, 0x014B + 4 * 0x10): | ||||
|         SyncLightDistanceAttenuationScale(4); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[5].dist_atten_scale, 0x014B + 5 * 0x10): | ||||
|         SyncLightDistanceAttenuationScale(5); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[6].dist_atten_scale, 0x014B + 6 * 0x10): | ||||
|         SyncLightDistanceAttenuationScale(6); | ||||
|         break; | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.light[7].dist_atten_scale, 0x014B + 7 * 0x10): | ||||
|         SyncLightDistanceAttenuationScale(7); | ||||
|         break; | ||||
| 
 | ||||
|     // Fragment lighting global ambient color (emission + ambient * ambient)
 | ||||
|     case PICA_REG_INDEX_WORKAROUND(lighting.global_ambient, 0x1c0): | ||||
|         SyncGlobalAmbient(); | ||||
|  | @ -896,6 +948,8 @@ void RasterizerOpenGL::SetShader() { | |||
|             SyncLightDiffuse(light_index); | ||||
|             SyncLightAmbient(light_index); | ||||
|             SyncLightPosition(light_index); | ||||
|             SyncLightDistanceAttenuationBias(light_index); | ||||
|             SyncLightDistanceAttenuationScale(light_index); | ||||
|         } | ||||
|     } | ||||
| } | ||||
|  | @ -1105,3 +1159,21 @@ void RasterizerOpenGL::SyncLightPosition(int light_index) { | |||
|         uniform_block_data.dirty = true; | ||||
|     } | ||||
| } | ||||
| 
 | ||||
| void RasterizerOpenGL::SyncLightDistanceAttenuationBias(int light_index) { | ||||
|     GLfloat dist_atten_bias = Pica::float20::FromRaw(Pica::g_state.regs.lighting.light[light_index].dist_atten_bias).ToFloat32(); | ||||
| 
 | ||||
|     if (dist_atten_bias != uniform_block_data.data.light_src[light_index].dist_atten_bias) { | ||||
|         uniform_block_data.data.light_src[light_index].dist_atten_bias = dist_atten_bias; | ||||
|         uniform_block_data.dirty = true; | ||||
|     } | ||||
| } | ||||
| 
 | ||||
| void RasterizerOpenGL::SyncLightDistanceAttenuationScale(int light_index) { | ||||
|     GLfloat dist_atten_scale = Pica::float20::FromRaw(Pica::g_state.regs.lighting.light[light_index].dist_atten_scale).ToFloat32(); | ||||
| 
 | ||||
|     if (dist_atten_scale != uniform_block_data.data.light_src[light_index].dist_atten_scale) { | ||||
|         uniform_block_data.data.light_src[light_index].dist_atten_scale = dist_atten_scale; | ||||
|         uniform_block_data.dirty = true; | ||||
|     } | ||||
| } | ||||
|  |  | |||
|  | @ -92,8 +92,6 @@ union PicaShaderConfig { | |||
|             state.lighting.light[light_index].directional = light.directional != 0; | ||||
|             state.lighting.light[light_index].two_sided_diffuse = light.two_sided_diffuse != 0; | ||||
|             state.lighting.light[light_index].dist_atten_enable = !regs.lighting.IsDistAttenDisabled(num); | ||||
|             state.lighting.light[light_index].dist_atten_bias = Pica::float20::FromRaw(light.dist_atten_bias).ToFloat32(); | ||||
|             state.lighting.light[light_index].dist_atten_scale = Pica::float20::FromRaw(light.dist_atten_scale).ToFloat32(); | ||||
|         } | ||||
| 
 | ||||
|         state.lighting.lut_d0.enable = regs.lighting.disable_lut_d0 == 0; | ||||
|  | @ -184,8 +182,6 @@ union PicaShaderConfig { | |||
|                 bool directional; | ||||
|                 bool two_sided_diffuse; | ||||
|                 bool dist_atten_enable; | ||||
|                 GLfloat dist_atten_scale; | ||||
|                 GLfloat dist_atten_bias; | ||||
|             } light[8]; | ||||
| 
 | ||||
|             bool enable; | ||||
|  | @ -316,6 +312,8 @@ private: | |||
|         alignas(16) GLvec3 diffuse; | ||||
|         alignas(16) GLvec3 ambient; | ||||
|         alignas(16) GLvec3 position; | ||||
|         GLfloat dist_atten_bias; | ||||
|         GLfloat dist_atten_scale; | ||||
|     }; | ||||
| 
 | ||||
|     /// Uniform structure for the Uniform Buffer Object, all members must be 16-byte aligned
 | ||||
|  | @ -330,7 +328,7 @@ private: | |||
|         LightSrc light_src[8]; | ||||
|     }; | ||||
| 
 | ||||
|     static_assert(sizeof(UniformData) == 0x310, "The size of the UniformData structure has changed, update the structure in the shader"); | ||||
|     static_assert(sizeof(UniformData) == 0x390, "The size of the UniformData structure has changed, update the structure in the shader"); | ||||
|     static_assert(sizeof(UniformData) < 16384, "UniformData structure must be less than 16kb as per the OpenGL spec"); | ||||
| 
 | ||||
|     /// Sets the OpenGL shader in accordance with the current PICA register state
 | ||||
|  | @ -402,6 +400,12 @@ private: | |||
|     /// Syncs the specified light's position to match the PICA register
 | ||||
|     void SyncLightPosition(int light_index); | ||||
| 
 | ||||
|     /// Syncs the specified light's distance attenuation bias to match the PICA register
 | ||||
|     void SyncLightDistanceAttenuationBias(int light_index); | ||||
| 
 | ||||
|     /// Syncs the specified light's distance attenuation scale to match the PICA register
 | ||||
|     void SyncLightDistanceAttenuationScale(int light_index); | ||||
| 
 | ||||
|     OpenGLState state; | ||||
| 
 | ||||
|     RasterizerCacheOpenGL res_cache; | ||||
|  |  | |||
|  | @ -439,9 +439,7 @@ static void WriteLighting(std::string& out, const PicaShaderConfig& config) { | |||
|         // If enabled, compute distance attenuation value
 | ||||
|         std::string dist_atten = "1.0"; | ||||
|         if (light_config.dist_atten_enable) { | ||||
|             std::string scale = std::to_string(light_config.dist_atten_scale); | ||||
|             std::string bias = std::to_string(light_config.dist_atten_bias); | ||||
|             std::string index = "(" + scale + " * length(-view - " + light_src + ".position) + " + bias + ")"; | ||||
|             std::string index = "(" + light_src + ".dist_atten_scale * length(-view - " + light_src + ".position) + " + light_src + ".dist_atten_bias)"; | ||||
|             index = "((clamp(" + index + ", 0.0, FLOAT_255)))"; | ||||
|             const unsigned lut_num = ((unsigned)Regs::LightingSampler::DistanceAttenuation + light_config.num); | ||||
|             dist_atten = GetLutValue((Regs::LightingSampler)lut_num, index); | ||||
|  | @ -549,6 +547,8 @@ struct LightSrc { | |||
|     vec3 diffuse; | ||||
|     vec3 ambient; | ||||
|     vec3 position; | ||||
|     float dist_atten_bias; | ||||
|     float dist_atten_scale; | ||||
| }; | ||||
| 
 | ||||
| layout (std140) uniform shader_data { | ||||
|  |  | |||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue