raphiaura/KytyPS5
1
0
Fork 0
mirror of https://github.com/KytyPS5/KytyPS5.git synced 2026-10-20 19:07:01 +02:00
Code Issues Packages Projects Releases Wiki Activity

shader: separate depth comparison and regular image descriptors

Browse Source 
Co-authored-by: Lordix <lordix253634@gmail.com>
Co-authored-by: fxpw <fxpw@outlook.com>
Co-authored-by: Mac <me@itsmemac.com>
This commit is contained in:
nmzik
2026-09-07 04:06:14 +04:00
parent 9d1fc8328f
commit 49e59e572e
4 changed files with 43 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/graphics/shader/recompiler/backend/spirv/spirvEmitterAnalysis.cpp
+2 -1
View File
@@ -158,7 +158,8 @@ uint32_t ImageType(EmitterState& state, const IR::ImageResource& image) {
} }
const auto& info = ImageDimensionInfoFor(image.dimension); const auto& info = ImageDimensionInfoFor(image.dimension);
return state.builder.Type(OpTypeImage, return state.builder.Type(OpTypeImage,
{ImageScalarType(state, image.numeric_class), info.spirv_dimension, 0, {ImageScalarType(state, image.numeric_class), info.spirv_dimension,
image.depth_compare ? 1u : 0u,
info.arrayed, info.multisampled, sampled, format}); info.arrayed, info.multisampled, sampled, format});
} }
src/graphics/shader/recompiler/ir/ShaderIR.h
+15 -9
View File
@@ -255,9 +255,10 @@ struct StageOutput {
bool operator==(const StageOutput& other) const = default; bool operator==(const StageOutput& other) const = default;
}; };
inline constexpr uint32_t FirstImageBinding = 1u; inline constexpr uint32_t FirstImageBinding = 1u;
inline constexpr uint32_t FirstStorageImageBinding = 22u; inline constexpr uint32_t FirstComparisonImageBinding = 22u;
inline constexpr uint32_t ImageBindingCount = 36u; inline constexpr uint32_t FirstStorageImageBinding = 29u;
inline constexpr uint32_t ImageBindingCount = 43u;
enum class DescriptorBindingKind : uint32_t { enum class DescriptorBindingKind : uint32_t {
Buffers = 0u, Buffers = 0u,
@@ -270,8 +271,8 @@ enum class DescriptorBindingKind : uint32_t {
Count, Count,
}; };
static_assert(static_cast<uint32_t>(DescriptorBindingKind::Samplers) == 37u); static_assert(static_cast<uint32_t>(DescriptorBindingKind::Samplers) == 44u);
static_assert(static_cast<uint32_t>(DescriptorBindingKind::Count) == 43u); static_assert(static_cast<uint32_t>(DescriptorBindingKind::Count) == 50u);
struct PushData { struct PushData {
static constexpr uint32_t DwordCount = 32; static constexpr uint32_t DwordCount = 32;
@@ -316,9 +317,9 @@ DescriptorBindingForImage(const ImageResource& image) {
constexpr uint32_t SampledFloatBinding = 1u; constexpr uint32_t SampledFloatBinding = 1u;
constexpr uint32_t SampledUintBinding = 8u; constexpr uint32_t SampledUintBinding = 8u;
constexpr uint32_t SampledSintBinding = 15u; constexpr uint32_t SampledSintBinding = 15u;
constexpr uint32_t StorageFloatBinding = 22u; constexpr uint32_t StorageFloatBinding = FirstStorageImageBinding;
constexpr uint32_t StorageUintBinding = 27u; constexpr uint32_t StorageUintBinding = StorageFloatBinding + 5u;
constexpr uint32_t AtomicUintBinding = 32u; constexpr uint32_t AtomicUintBinding = StorageUintBinding + 5u;
uint32_t base = 0; uint32_t base = 0;
bool sampled = false; bool sampled = false;
@@ -328,12 +329,17 @@ DescriptorBindingForImage(const ImageResource& image) {
} }
sampled = true; sampled = true;
switch (image.numeric_class) { switch (image.numeric_class) {
case Prospero::TextureNumericClass::Float: base = SampledFloatBinding; break; case Prospero::TextureNumericClass::Float:
base = image.depth_compare ? FirstComparisonImageBinding : SampledFloatBinding;
break;
case Prospero::TextureNumericClass::Uint: base = SampledUintBinding; break; case Prospero::TextureNumericClass::Uint: base = SampledUintBinding; break;
case Prospero::TextureNumericClass::Sint: base = SampledSintBinding; break; case Prospero::TextureNumericClass::Sint: base = SampledSintBinding; break;
case Prospero::TextureNumericClass::Unsupported: return std::nullopt; case Prospero::TextureNumericClass::Unsupported: return std::nullopt;
default: return std::nullopt; default: return std::nullopt;
} }
if (image.depth_compare && image.numeric_class != Prospero::TextureNumericClass::Float) {
return std::nullopt;
}
} else if (image.resource_class == ImageResourceClass::Storage) { } else if (image.resource_class == ImageResourceClass::Storage) {
if (image.atomic) { if (image.atomic) {
if (image.numeric_class != Prospero::TextureNumericClass::Uint) { if (image.numeric_class != Prospero::TextureNumericClass::Uint) {
tests/ResourceTrackingTests.cpp
+18 -9
View File
@@ -1617,15 +1617,15 @@ void TestShaderInfoAndBindingLayout() {
void TestImageBindingAbi() { void TestImageBindingAbi() {
using NumericClass = Libs::Graphics::Prospero::TextureNumericClass; using NumericClass = Libs::Graphics::Prospero::TextureNumericClass;
Check(ImageBindingCount == 36u && Check(ImageBindingCount == 43u &&
static_cast<uint32_t>(DescriptorBindingKind::Buffers) == 0u && static_cast<uint32_t>(DescriptorBindingKind::Buffers) == 0u &&
static_cast<uint32_t>(DescriptorBindingKind::Samplers) == 37u && static_cast<uint32_t>(DescriptorBindingKind::Samplers) == 44u &&
static_cast<uint32_t>(DescriptorBindingKind::Gds) == 38u && static_cast<uint32_t>(DescriptorBindingKind::Gds) == 45u &&
static_cast<uint32_t>(DescriptorBindingKind::BdaPagetable) == 39u && static_cast<uint32_t>(DescriptorBindingKind::BdaPagetable) == 46u &&
static_cast<uint32_t>(DescriptorBindingKind::FaultBuffer) == 40u && static_cast<uint32_t>(DescriptorBindingKind::FaultBuffer) == 47u &&
static_cast<uint32_t>(DescriptorBindingKind::FlattenedSrt) == 41u && static_cast<uint32_t>(DescriptorBindingKind::FlattenedSrt) == 48u &&
static_cast<uint32_t>(DescriptorBindingKind::ShaderData) == 42u && static_cast<uint32_t>(DescriptorBindingKind::ShaderData) == 49u &&
static_cast<uint32_t>(DescriptorBindingKind::Count) == 43u, static_cast<uint32_t>(DescriptorBindingKind::Count) == 50u,
"native descriptor binding anchors changed"); "native descriptor binding anchors changed");
const std::array sampled_dimensions{ const std::array sampled_dimensions{
@@ -1648,12 +1648,13 @@ void TestImageBindingAbi() {
uint32_t index = 0; uint32_t index = 0;
const auto CheckBinding = const auto CheckBinding =
[&](ImageResourceClass resource_class, NumericClass numeric_class, [&](ImageResourceClass resource_class, NumericClass numeric_class,
Decoder::ImageDimension dimension, bool atomic) { Decoder::ImageDimension dimension, bool atomic, bool comparison = false) {
ImageResource image; ImageResource image;
image.resource_class = resource_class; image.resource_class = resource_class;
image.numeric_class = numeric_class; image.numeric_class = numeric_class;
image.dimension = dimension; image.dimension = dimension;
image.atomic = atomic; image.atomic = atomic;
image.depth_compare = comparison;
const auto kind = DescriptorBindingForImage(image); const auto kind = DescriptorBindingForImage(image);
Check(kind.has_value() && Check(kind.has_value() &&
static_cast<uint32_t>(*kind) == FirstImageBinding + index && static_cast<uint32_t>(*kind) == FirstImageBinding + index &&
@@ -1673,6 +1674,10 @@ void TestImageBindingAbi() {
false); false);
} }
} }
for (const auto dimension : sampled_dimensions) {
CheckBinding(ImageResourceClass::Sampled, NumericClass::Float, dimension,
false, true);
}
for (const auto numeric_class : storage_classes) { for (const auto numeric_class : storage_classes) {
for (const auto dimension : storage_dimensions) { for (const auto dimension : storage_dimensions) {
CheckBinding(ImageResourceClass::Storage, numeric_class, dimension, CheckBinding(ImageResourceClass::Storage, numeric_class, dimension,
@@ -1699,6 +1704,10 @@ void TestImageBindingAbi() {
image.numeric_class = NumericClass::Unsupported; image.numeric_class = NumericClass::Unsupported;
Check(Invalid(image), Check(Invalid(image),
"unsupported sampled class received a descriptor binding"); "unsupported sampled class received a descriptor binding");
image.numeric_class = NumericClass::Uint;
image.depth_compare = true;
Check(Invalid(image), "integer comparison image received a descriptor binding");
image.depth_compare = false;
image.numeric_class = static_cast<NumericClass>(UINT32_MAX); image.numeric_class = static_cast<NumericClass>(UINT32_MAX);
Check(Invalid(image), "invalid sampled class received a descriptor binding"); Check(Invalid(image), "invalid sampled class received a descriptor binding");
image.numeric_class = NumericClass::Float; image.numeric_class = NumericClass::Float;
tests/ShaderRecompilerComputeTests.cpp
+8 -1
View File
@@ -22307,6 +22307,10 @@ TestCase ImageSampleA16CompareBiasRdna2AddressOrder() {
code.push_back(EncodeMimg0(0x2d, 0x1)); code.push_back(EncodeMimg0(0x2d, 0x1));
code.push_back(EncodeMimg1(0, 20, 0, 0, true)); code.push_back(EncodeMimg1(0, 20, 0, 0, true));
AppendStoreVgpr(&code, 0, 0); AppendStoreVgpr(&code, 0, 0);
// Ordinary and comparison reads of one sharp require distinct SPIR-V image types.
code.push_back(EncodeMimg0(0x27, 0x1));
code.push_back(EncodeMimg1(1, 22, 0, 0, true));
AppendStoreVgpr(&code, 1, 1);
AppendEnd(&code); AppendEnd(&code);
TestCase test; TestCase test;
@@ -22314,7 +22318,9 @@ TestCase ImageSampleA16CompareBiasRdna2AddressOrder() {
test.code = code; test.code = code;
test.opcodes = {O::V_MOV_B32, O::IMAGE_SAMPLE, O::BUFFER_STORE_DWORD, test.opcodes = {O::V_MOV_B32, O::IMAGE_SAMPLE, O::BUFFER_STORE_DWORD,
O::S_ENDPGM}; O::S_ENDPGM};
test.required_spirv = {"OpImageSampleDrefExplicitLod", "UnpackHalf2x16"}; test.required_spirv = {"OpTypeImage %float 2D 0", "OpTypeImage %float 2D 1",
"OpImageSampleDrefExplicitLod", "OpImageSampleExplicitLod",
"UnpackHalf2x16"};
test.compile_only = true; test.compile_only = true;
return test; return test;
} }
@@ -28097,6 +28103,7 @@ int main(int argc, char **argv) {
CheckSampledDepthResource(); CheckSampledDepthResource();
CheckDepthTextureEncoding(vulkan.RuntimeRenderer()); CheckDepthTextureEncoding(vulkan.RuntimeRenderer());
vulkan.CheckComparisonDepthTexture(); vulkan.CheckComparisonDepthTexture();
RunCase(nullptr, ImageSampleA16CompareBiasRdna2AddressOrder());
return 0; return 0;
} }
#if KYTY_PLATFORM == KYTY_PLATFORM_WINDOWS #if KYTY_PLATFORM == KYTY_PLATFORM_WINDOWS