Merge pull request #8873 from vonchenplus/fix_legacy_location_error

video_core: Fix legacy to generic location unpaired
This commit is contained in:
bunnei 2022-10-24 10:50:24 -07:00 committed by GitHub
commit cbb6c24215
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
5 changed files with 35 additions and 19 deletions

View File

@ -137,28 +137,35 @@ bool IsLegacyAttribute(IR::Attribute attribute) {
} }
std::map<IR::Attribute, IR::Attribute> GenerateLegacyToGenericMappings( std::map<IR::Attribute, IR::Attribute> GenerateLegacyToGenericMappings(
const VaryingState& state, std::queue<IR::Attribute> ununsed_generics) { const VaryingState& state, std::queue<IR::Attribute> unused_generics,
const std::map<IR::Attribute, IR::Attribute>& previous_stage_mapping) {
std::map<IR::Attribute, IR::Attribute> mapping; std::map<IR::Attribute, IR::Attribute> mapping;
auto update_mapping = [&mapping, &unused_generics, previous_stage_mapping](IR::Attribute attr,
size_t count) {
if (previous_stage_mapping.find(attr) != previous_stage_mapping.end()) {
for (size_t i = 0; i < count; ++i) {
mapping.insert({attr + i, previous_stage_mapping.at(attr + i)});
}
} else {
for (size_t i = 0; i < count; ++i) {
mapping.insert({attr + i, unused_generics.front() + i});
}
unused_generics.pop();
}
};
for (size_t index = 0; index < 4; ++index) { for (size_t index = 0; index < 4; ++index) {
auto attr = IR::Attribute::ColorFrontDiffuseR + index * 4; auto attr = IR::Attribute::ColorFrontDiffuseR + index * 4;
if (state.AnyComponent(attr)) { if (state.AnyComponent(attr)) {
for (size_t i = 0; i < 4; ++i) { update_mapping(attr, 4);
mapping.insert({attr + i, ununsed_generics.front() + i});
}
ununsed_generics.pop();
} }
} }
if (state[IR::Attribute::FogCoordinate]) { if (state[IR::Attribute::FogCoordinate]) {
mapping.insert({IR::Attribute::FogCoordinate, ununsed_generics.front()}); update_mapping(IR::Attribute::FogCoordinate, 1);
ununsed_generics.pop();
} }
for (size_t index = 0; index < IR::NUM_FIXEDFNCTEXTURE; ++index) { for (size_t index = 0; index < IR::NUM_FIXEDFNCTEXTURE; ++index) {
auto attr = IR::Attribute::FixedFncTexture0S + index * 4; auto attr = IR::Attribute::FixedFncTexture0S + index * 4;
if (state.AnyComponent(attr)) { if (state.AnyComponent(attr)) {
for (size_t i = 0; i < 4; ++i) { update_mapping(attr, 4);
mapping.insert({attr + i, ununsed_generics.front() + i});
}
ununsed_generics.pop();
} }
} }
return mapping; return mapping;
@ -265,21 +272,22 @@ IR::Program MergeDualVertexPrograms(IR::Program& vertex_a, IR::Program& vertex_b
void ConvertLegacyToGeneric(IR::Program& program, const Shader::RuntimeInfo& runtime_info) { void ConvertLegacyToGeneric(IR::Program& program, const Shader::RuntimeInfo& runtime_info) {
auto& stores = program.info.stores; auto& stores = program.info.stores;
if (stores.Legacy()) { if (stores.Legacy()) {
std::queue<IR::Attribute> ununsed_output_generics{}; std::queue<IR::Attribute> unused_output_generics{};
for (size_t index = 0; index < IR::NUM_GENERICS; ++index) { for (size_t index = 0; index < IR::NUM_GENERICS; ++index) {
if (!stores.Generic(index)) { if (!stores.Generic(index)) {
ununsed_output_generics.push(IR::Attribute::Generic0X + index * 4); unused_output_generics.push(IR::Attribute::Generic0X + index * 4);
} }
} }
auto mappings = GenerateLegacyToGenericMappings(stores, ununsed_output_generics); program.info.legacy_stores_mapping =
GenerateLegacyToGenericMappings(stores, unused_output_generics, {});
for (IR::Block* const block : program.post_order_blocks) { for (IR::Block* const block : program.post_order_blocks) {
for (IR::Inst& inst : block->Instructions()) { for (IR::Inst& inst : block->Instructions()) {
switch (inst.GetOpcode()) { switch (inst.GetOpcode()) {
case IR::Opcode::SetAttribute: { case IR::Opcode::SetAttribute: {
const auto attr = inst.Arg(0).Attribute(); const auto attr = inst.Arg(0).Attribute();
if (IsLegacyAttribute(attr)) { if (IsLegacyAttribute(attr)) {
stores.Set(mappings[attr], true); stores.Set(program.info.legacy_stores_mapping[attr], true);
inst.SetArg(0, Shader::IR::Value(mappings[attr])); inst.SetArg(0, Shader::IR::Value(program.info.legacy_stores_mapping[attr]));
} }
break; break;
} }
@ -292,15 +300,16 @@ void ConvertLegacyToGeneric(IR::Program& program, const Shader::RuntimeInfo& run
auto& loads = program.info.loads; auto& loads = program.info.loads;
if (loads.Legacy()) { if (loads.Legacy()) {
std::queue<IR::Attribute> ununsed_input_generics{}; std::queue<IR::Attribute> unused_input_generics{};
for (size_t index = 0; index < IR::NUM_GENERICS; ++index) { for (size_t index = 0; index < IR::NUM_GENERICS; ++index) {
const AttributeType input_type{runtime_info.generic_input_types[index]}; const AttributeType input_type{runtime_info.generic_input_types[index]};
if (!runtime_info.previous_stage_stores.Generic(index) || !loads.Generic(index) || if (!runtime_info.previous_stage_stores.Generic(index) || !loads.Generic(index) ||
input_type == AttributeType::Disabled) { input_type == AttributeType::Disabled) {
ununsed_input_generics.push(IR::Attribute::Generic0X + index * 4); unused_input_generics.push(IR::Attribute::Generic0X + index * 4);
} }
} }
auto mappings = GenerateLegacyToGenericMappings(loads, ununsed_input_generics); auto mappings = GenerateLegacyToGenericMappings(
loads, unused_input_generics, runtime_info.previous_stage_legacy_stores_mapping);
for (IR::Block* const block : program.post_order_blocks) { for (IR::Block* const block : program.post_order_blocks) {
for (IR::Inst& inst : block->Instructions()) { for (IR::Inst& inst : block->Instructions()) {
switch (inst.GetOpcode()) { switch (inst.GetOpcode()) {

View File

@ -4,6 +4,7 @@
#pragma once #pragma once
#include <array> #include <array>
#include <map>
#include <optional> #include <optional>
#include <vector> #include <vector>
@ -60,6 +61,7 @@ struct TransformFeedbackVarying {
struct RuntimeInfo { struct RuntimeInfo {
std::array<AttributeType, 32> generic_input_types{}; std::array<AttributeType, 32> generic_input_types{};
VaryingState previous_stage_stores; VaryingState previous_stage_stores;
std::map<IR::Attribute, IR::Attribute> previous_stage_legacy_stores_mapping;
bool convert_depth_mode{}; bool convert_depth_mode{};
bool force_early_z{}; bool force_early_z{};

View File

@ -5,6 +5,7 @@
#include <array> #include <array>
#include <bitset> #include <bitset>
#include <map>
#include "common/common_types.h" #include "common/common_types.h"
#include "shader_recompiler/frontend/ir/type.h" #include "shader_recompiler/frontend/ir/type.h"
@ -127,6 +128,8 @@ struct Info {
VaryingState stores; VaryingState stores;
VaryingState passthrough; VaryingState passthrough;
std::map<IR::Attribute, IR::Attribute> legacy_stores_mapping;
bool loads_indexed_attributes{}; bool loads_indexed_attributes{};
std::array<bool, 8> stores_frag_color{}; std::array<bool, 8> stores_frag_color{};

View File

@ -63,6 +63,7 @@ Shader::RuntimeInfo MakeRuntimeInfo(const GraphicsPipelineKey& key,
Shader::RuntimeInfo info; Shader::RuntimeInfo info;
if (previous_program) { if (previous_program) {
info.previous_stage_stores = previous_program->info.stores; info.previous_stage_stores = previous_program->info.stores;
info.previous_stage_legacy_stores_mapping = previous_program->info.legacy_stores_mapping;
} else { } else {
// Mark all stores as available for vertex shaders // Mark all stores as available for vertex shaders
info.previous_stage_stores.mask.set(); info.previous_stage_stores.mask.set();

View File

@ -134,6 +134,7 @@ Shader::RuntimeInfo MakeRuntimeInfo(std::span<const Shader::IR::Program> program
Shader::RuntimeInfo info; Shader::RuntimeInfo info;
if (previous_program) { if (previous_program) {
info.previous_stage_stores = previous_program->info.stores; info.previous_stage_stores = previous_program->info.stores;
info.previous_stage_legacy_stores_mapping = previous_program->info.legacy_stores_mapping;
if (previous_program->is_geometry_passthrough) { if (previous_program->is_geometry_passthrough) {
info.previous_stage_stores.mask |= previous_program->info.passthrough.mask; info.previous_stage_stores.mask |= previous_program->info.passthrough.mask;
} }