Browse Source

shader: Address feedback

nce_cpp
FernandoS27 5 years ago
committed by ameerj
parent
commit
f1a3f449d6
  1. 2
      src/shader_recompiler/backend/spirv/emit_spirv.cpp
  2. 5
      src/shader_recompiler/backend/spirv/emit_spirv_context_get_set.cpp
  3. 2
      src/shader_recompiler/ir_opt/collect_shader_info_pass.cpp
  4. 2
      src/shader_recompiler/shader_info.h

2
src/shader_recompiler/backend/spirv/emit_spirv.cpp

@ -341,7 +341,7 @@ void SetupCapabilities(const Profile& profile, const Info& info, EmitContext& ct
if (!ctx.profile.xfb_varyings.empty()) {
ctx.AddCapability(spv::Capability::TransformFeedback);
}
if (info.uses_derivates) {
if (info.uses_derivatives) {
ctx.AddCapability(spv::Capability::DerivativeControl);
}
// TODO: Track this usage

5
src/shader_recompiler/backend/spirv/emit_spirv_context_get_set.cpp

@ -404,10 +404,7 @@ Id EmitIsHelperInvocation(EmitContext& ctx) {
}
Id EmitYDirection(EmitContext& ctx) {
if (ctx.profile.y_negate) {
return ctx.Constant(ctx.F32[1], -1.0f);
}
return ctx.Constant(ctx.F32[1], 1.0f);
return ctx.Constant(ctx.F32[1], ctx.profile.y_negate ? -1.0f : 1.0f);
}
Id EmitLoadLocal(EmitContext& ctx, Id word_offset) {

2
src/shader_recompiler/ir_opt/collect_shader_info_pass.cpp

@ -534,7 +534,7 @@ void VisitUsages(Info& info, IR::Inst& inst) {
case IR::Opcode::DPdyFine:
case IR::Opcode::DPdxCoarse:
case IR::Opcode::DPdyCoarse:
info.uses_derivates = true;
info.uses_derivatives = true;
break;
case IR::Opcode::LoadStorageU8:
case IR::Opcode::LoadStorageS8:

2
src/shader_recompiler/shader_info.h

@ -147,7 +147,7 @@ struct Info {
bool uses_subgroup_vote{};
bool uses_subgroup_mask{};
bool uses_fswzadd{};
bool uses_derivates{};
bool uses_derivatives{};
bool uses_typeless_image_reads{};
bool uses_typeless_image_writes{};
bool uses_shared_increment{};

Loading…
Cancel
Save