aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorryana <ryanamayque@gmail.com>2026-05-04 18:16:11 +0200
committercrueter <crueter@eden-emu.dev>2026-05-04 18:16:11 +0200
commit27e5cb0f126749d27935215a3c67ec338a0f06fc (patch)
treec4363f6bbb9a9b09477814c436f90adb012a7868 /src
parenta76c76d122129da7399cd9b6a5d9d6695a95e8cf (diff)
[spirv] mark sampled image descriptor indices non-uniform (#3900)
fixes incorrect texture selection on vk when shaders use per-pixel descriptor indices, in line with #3898 so dynamic descs are no longer treated as uniform also fixes TD;LTD spotty grass issue on SD not addressed by above pr you can test out all the fixes here: https://git.eden-emu.dev/may/eden/src/branch/may/integrate-texture-descriptor-fixes Reviewed-on: https://git.eden-emu.dev/eden-emu/eden/pulls/3900 Reviewed-by: crueter <crueter@eden-emu.dev>
Diffstat (limited to 'src')
-rw-r--r--src/shader_recompiler/backend/spirv/emit_spirv_image.cpp51
-rw-r--r--src/shader_recompiler/profile.h4
-rw-r--r--src/video_core/renderer_vulkan/vk_pipeline_cache.cpp2
-rw-r--r--src/video_core/vulkan_common/vulkan_device.h5
4 files changed, 57 insertions, 5 deletions
diff --git a/src/shader_recompiler/backend/spirv/emit_spirv_image.cpp b/src/shader_recompiler/backend/spirv/emit_spirv_image.cpp
index c2511942d9..62a82dce9d 100644
--- a/src/shader_recompiler/backend/spirv/emit_spirv_image.cpp
+++ b/src/shader_recompiler/backend/spirv/emit_spirv_image.cpp
@@ -14,6 +14,38 @@
namespace Shader::Backend::SPIRV {
namespace {
+class DescriptorIndex {
+public:
+ explicit DescriptorIndex(EmitContext& ctx, const IR::Value& index)
+ : id{index.IsImmediate() ? ctx.Const(index.U32()) : ctx.Def(index)},
+ is_non_uniform{ctx.profile.support_sampled_image_array_nonuniform_indexing &&
+ !index.IsImmediate()} {
+ if (!is_non_uniform) {
+ return;
+ }
+ if (ctx.profile.supported_spirv < 0x00010400) {
+ ctx.AddExtension("SPV_EXT_descriptor_indexing");
+ }
+ ctx.AddCapability(spv::Capability::ShaderNonUniform);
+ ctx.AddCapability(spv::Capability::SampledImageArrayNonUniformIndexing);
+ Decorate(ctx, id);
+ }
+
+ Id Value() const {
+ return id;
+ }
+
+ void Decorate(EmitContext& ctx, Id object) const {
+ if (is_non_uniform) {
+ ctx.Decorate(object, spv::Decoration::NonUniform);
+ }
+ }
+
+private:
+ Id id;
+ bool is_non_uniform;
+};
+
class ImageOperands {
public:
[[maybe_unused]] static constexpr bool ImageSampleOffsetAllowed = false;
@@ -189,8 +221,12 @@ private:
Id Texture(EmitContext& ctx, IR::TextureInstInfo info, [[maybe_unused]] const IR::Value& index) {
const TextureDefinition& def{ctx.textures.at(info.descriptor_index)};
if (def.count > 1) {
- const Id pointer{ctx.OpAccessChain(def.pointer_type, def.id, ctx.Def(index))};
- return ctx.OpLoad(def.sampled_type, pointer);
+ const DescriptorIndex idx{ctx, index};
+ const Id pointer{ctx.OpAccessChain(def.pointer_type, def.id, idx.Value())};
+ idx.Decorate(ctx, pointer);
+ const Id object{ctx.OpLoad(def.sampled_type, pointer)};
+ idx.Decorate(ctx, object);
+ return object;
} else {
return ctx.OpLoad(def.sampled_type, def.id);
}
@@ -208,9 +244,14 @@ Id TextureImage(EmitContext& ctx, IR::TextureInstInfo info, const IR::Value& ind
} else {
const TextureDefinition& def{ctx.textures.at(info.descriptor_index)};
if (def.count > 1) {
- const Id idx{index.IsImmediate() ? ctx.Const(index.U32()) : ctx.Def(index)};
- const Id ptr{ctx.OpAccessChain(def.pointer_type, def.id, idx)};
- return ctx.OpImage(def.image_type, ctx.OpLoad(def.sampled_type, ptr));
+ const DescriptorIndex idx{ctx, index};
+ const Id ptr{ctx.OpAccessChain(def.pointer_type, def.id, idx.Value())};
+ idx.Decorate(ctx, ptr);
+ const Id object{ctx.OpLoad(def.sampled_type, ptr)};
+ idx.Decorate(ctx, object);
+ const Id image{ctx.OpImage(def.image_type, object)};
+ idx.Decorate(ctx, image);
+ return image;
}
return ctx.OpImage(def.image_type, ctx.OpLoad(def.sampled_type, def.id));
}
diff --git a/src/shader_recompiler/profile.h b/src/shader_recompiler/profile.h
index 90e46bb1ba..ff19f0710f 100644
--- a/src/shader_recompiler/profile.h
+++ b/src/shader_recompiler/profile.h
@@ -1,3 +1,6 @@
+// SPDX-FileCopyrightText: Copyright 2026 Eden Emulator Project
+// SPDX-License-Identifier: GPL-3.0-or-later
+
// SPDX-FileCopyrightText: Copyright 2021 yuzu Emulator Project
// SPDX-License-Identifier: GPL-2.0-or-later
@@ -45,6 +48,7 @@ struct Profile {
bool support_scaled_attributes{};
bool support_multi_viewport{};
bool support_geometry_streams{};
+ bool support_sampled_image_array_nonuniform_indexing{};
bool warp_size_potentially_larger_than_guest{};
diff --git a/src/video_core/renderer_vulkan/vk_pipeline_cache.cpp b/src/video_core/renderer_vulkan/vk_pipeline_cache.cpp
index 087e985892..e365425600 100644
--- a/src/video_core/renderer_vulkan/vk_pipeline_cache.cpp
+++ b/src/video_core/renderer_vulkan/vk_pipeline_cache.cpp
@@ -412,6 +412,8 @@ PipelineCache::PipelineCache(Tegra::MaxwellDeviceMemoryManager& device_memory_,
.support_scaled_attributes = !device.MustEmulateScaledFormats(),
.support_multi_viewport = device.SupportsMultiViewport(),
.support_geometry_streams = device.AreTransformFeedbackGeometryStreamsSupported(),
+ .support_sampled_image_array_nonuniform_indexing =
+ device.IsSampledImageArrayNonUniformIndexingSupported(),
.warp_size_potentially_larger_than_guest = device.IsWarpSizePotentiallyBiggerThanGuest(),
diff --git a/src/video_core/vulkan_common/vulkan_device.h b/src/video_core/vulkan_common/vulkan_device.h
index a5ffd08eac..13fe1b371f 100644
--- a/src/video_core/vulkan_common/vulkan_device.h
+++ b/src/video_core/vulkan_common/vulkan_device.h
@@ -33,6 +33,7 @@ VK_DEFINE_HANDLE(VmaAllocator)
FEATURE(KHR, VariablePointer, VARIABLE_POINTERS, variable_pointer)
#define FOR_EACH_VK_FEATURE_1_2(FEATURE) \
+ FEATURE(EXT, DescriptorIndexing, DESCRIPTOR_INDEXING, descriptor_indexing) \
FEATURE(EXT, HostQueryReset, HOST_QUERY_RESET, host_query_reset) \
FEATURE(KHR, 8BitStorage, 8BIT_STORAGE, bit8_storage) \
FEATURE(KHR, TimelineSemaphore, TIMELINE_SEMAPHORE, timeline_semaphore)
@@ -367,6 +368,10 @@ public:
return GetDriverID() != VK_DRIVER_ID_QUALCOMM_PROPRIETARY;
}
+ bool IsSampledImageArrayNonUniformIndexingSupported() const {
+ return features.descriptor_indexing.shaderSampledImageArrayNonUniformIndexing;
+ }
+
/// Returns true if the device supports float64 natively.
bool IsFloat64Supported() const {
return features.features.shaderFloat64;