Bind TextureBuffers

Currently unimplemented and issues are arising with building BindingSegments in general.

src/Ryujinx.Graphics.Metal/EncoderState.cs

@@ -54,10 +54,10 @@ namespace Ryujinx.Graphics.Metal
     record struct TextureRef
     {
         public ShaderStage Stage;
-        public Texture Storage;
+        public TextureBase Storage;
         public Sampler Sampler;
 
-        public TextureRef(ShaderStage stage, Texture storage, Sampler sampler)
+        public TextureRef(ShaderStage stage, TextureBase storage, Sampler sampler)
         {
             Stage = stage;
             Storage = storage;

src/Ryujinx.Graphics.Metal/EncoderStateManager.cs

@@ -805,13 +805,9 @@ namespace Ryujinx.Graphics.Metal
 
         public readonly void UpdateTextureAndSampler(ShaderStage stage, ulong binding, TextureBase texture, Sampler sampler)
         {
-            if (texture is TextureBuffer)
+            if (texture != null)
             {
-                // TODO: Texture buffers
+                _currentState.TextureRefs[binding] = new(stage, texture, sampler);
-            }
-            else if (texture is Texture view)
-            {
-                _currentState.TextureRefs[binding] = new(stage, view, sampler);
             }
             else
             {
@@ -1124,59 +1120,52 @@ namespace Ryujinx.Graphics.Metal
                     case MetalRenderer.TextureSetIndex:
                         if (!segment.IsArray)
                         {
-                            if (segment.Type != ResourceType.BufferTexture)
+                            for (int i = 0; i < count; i++)
                             {
-                                for (int i = 0; i < count; i++)
+                                int index = binding + i;
+
+                                ref var texture = ref _currentState.TextureRefs[index];
+
+                                var storage = texture.Storage;
+
+                                if (storage == null)
                                 {
-                                    int index = binding + i;
+                                    continue;
-
-                                    ref var texture = ref _currentState.TextureRefs[index];
-
-                                    var storage = texture.Storage;
-
-                                    if (storage == null)
-                                    {
-                                        continue;
-                                    }
-
-                                    var mtlTexture = storage.GetHandle();
-
-                                    MTLRenderStages renderStages = 0;
-
-                                    if ((segment.Stages & ResourceStages.Vertex) != 0)
-                                    {
-                                        vertResourceIds[vertResourceIdIndex] = mtlTexture.GpuResourceID._impl;
-                                        vertResourceIdIndex++;
-
-                                        if (texture.Sampler != null)
-                                        {
-                                            vertResourceIds[vertResourceIdIndex] = texture.Sampler.GetSampler().GpuResourceID._impl;
-                                            vertResourceIdIndex++;
-                                        }
-
-                                        renderStages |= MTLRenderStages.RenderStageVertex;
-                                    }
-
-                                    if ((segment.Stages & ResourceStages.Fragment) != 0)
-                                    {
-                                        fragResourceIds[fragResourceIdIndex] = mtlTexture.GpuResourceID._impl;
-                                        fragResourceIdIndex++;
-
-                                        if (texture.Sampler != null)
-                                        {
-                                            fragResourceIds[fragResourceIdIndex] = texture.Sampler.GetSampler().GpuResourceID._impl;
-                                            fragResourceIdIndex++;
-                                        }
-
-                                        renderStages |= MTLRenderStages.RenderStageFragment;
-                                    }
-
-                                    renderCommandEncoder.UseResource(new MTLResource(mtlTexture.NativePtr), MTLResourceUsage.Read, renderStages);
                                 }
-                            }
+
-                            else
+                                var mtlTexture = storage.GetHandle();
-                            {
+
-                                // TODO: Buffer textures
+                                MTLRenderStages renderStages = 0;
+
+                                if ((segment.Stages & ResourceStages.Vertex) != 0)
+                                {
+                                    vertResourceIds[vertResourceIdIndex] = mtlTexture.GpuResourceID._impl;
+                                    vertResourceIdIndex++;
+
+                                    if (texture.Sampler != null)
+                                    {
+                                        vertResourceIds[vertResourceIdIndex] = texture.Sampler.GetSampler().GpuResourceID._impl;
+                                        vertResourceIdIndex++;
+                                    }
+
+                                    renderStages |= MTLRenderStages.RenderStageVertex;
+                                }
+
+                                if ((segment.Stages & ResourceStages.Fragment) != 0)
+                                {
+                                    fragResourceIds[fragResourceIdIndex] = mtlTexture.GpuResourceID._impl;
+                                    fragResourceIdIndex++;
+
+                                    if (texture.Sampler != null)
+                                    {
+                                        fragResourceIds[fragResourceIdIndex] = texture.Sampler.GetSampler().GpuResourceID._impl;
+                                        fragResourceIdIndex++;
+                                    }
+
+                                    renderStages |= MTLRenderStages.RenderStageFragment;
+                                }
+
+                                renderCommandEncoder.UseResource(new MTLResource(mtlTexture.NativePtr), MTLResourceUsage.Read, renderStages);
                             }
                         }
                         else
@@ -1343,41 +1332,34 @@ namespace Ryujinx.Graphics.Metal
                     case MetalRenderer.TextureSetIndex:
                         if (!segment.IsArray)
                         {
-                            if (segment.Type != ResourceType.BufferTexture)
+                            for (int i = 0; i < count; i++)
                             {
-                                for (int i = 0; i < count; i++)
+                                int index = binding + i;
+
+                                ref var texture = ref _currentState.TextureRefs[index];
+
+                                var storage = texture.Storage;
+
+                                if (storage == null)
                                 {
-                                    int index = binding + i;
+                                    continue;
+                                }
 
-                                    ref var texture = ref _currentState.TextureRefs[index];
+                                var mtlTexture = storage.GetHandle();
 
-                                    var storage = texture.Storage;
+                                if (segment.Stages.HasFlag(ResourceStages.Compute))
+                                {
+                                    computeCommandEncoder.UseResource(new MTLResource(mtlTexture.NativePtr), MTLResourceUsage.Read);
+                                    resourceIds[resourceIdIndex] = mtlTexture.GpuResourceID._impl;
+                                    resourceIdIndex++;
 
-                                    if (storage == null)
+                                    if (texture.Sampler != null)
                                     {
-                                        continue;
+                                        resourceIds[resourceIdIndex] = texture.Sampler.GetSampler().GpuResourceID._impl;
-                                    }
-
-                                    var mtlTexture = storage.GetHandle();
-
-                                    if (segment.Stages.HasFlag(ResourceStages.Compute))
-                                    {
-                                        computeCommandEncoder.UseResource(new MTLResource(mtlTexture.NativePtr), MTLResourceUsage.Read);
-                                        resourceIds[resourceIdIndex] = mtlTexture.GpuResourceID._impl;
                                         resourceIdIndex++;
-
-                                        if (texture.Sampler != null)
-                                        {
-                                            resourceIds[resourceIdIndex] = texture.Sampler.GetSampler().GpuResourceID._impl;
-                                            resourceIdIndex++;
-                                        }
                                     }
                                 }
                             }
-                            else
-                            {
-                                // TODO: Buffer textures
-                            }
                         }
                         else
                         {

src/Ryujinx.Graphics.Metal/Program.cs

@@ -28,7 +28,6 @@ namespace Ryujinx.Graphics.Metal
         private MTLComputePipelineState? _computePipelineCache;
         private bool _firstBackgroundUse;
 
-        public ResourceBindingSegment[][] ClearSegments { get; }
         public ResourceBindingSegment[][] BindingSegments { get; }
         // Argument buffer sizes for Vertex or Compute stages
         public int[] ArgumentBufferSizes { get; }
@@ -53,7 +52,6 @@ namespace Ryujinx.Graphics.Metal
                 _handles[i] = device.NewLibrary(StringHelper.NSString(shader.Code), compileOptions, (library, error) => CompilationResultHandler(library, error, index));
             }
 
-            ClearSegments = BuildClearSegments(resourceLayout.Sets);
             (BindingSegments, ArgumentBufferSizes, FragArgumentBufferSizes) = BuildBindingSegments(resourceLayout.SetUsages);
         }
 
@@ -98,62 +96,6 @@ namespace Ryujinx.Graphics.Metal
             }
         }
 
-        private static ResourceBindingSegment[][] BuildClearSegments(ReadOnlyCollection<ResourceDescriptorCollection> sets)
-        {
-            ResourceBindingSegment[][] segments = new ResourceBindingSegment[sets.Count][];
-
-            for (int setIndex = 0; setIndex < sets.Count; setIndex++)
-            {
-                List<ResourceBindingSegment> currentSegments = new();
-
-                ResourceDescriptor currentDescriptor = default;
-                int currentCount = 0;
-
-                for (int index = 0; index < sets[setIndex].Descriptors.Count; index++)
-                {
-                    ResourceDescriptor descriptor = sets[setIndex].Descriptors[index];
-
-                    if (currentDescriptor.Binding + currentCount != descriptor.Binding ||
-                        currentDescriptor.Type != descriptor.Type ||
-                        currentDescriptor.Stages != descriptor.Stages ||
-                        currentDescriptor.Count > 1 ||
-                        descriptor.Count > 1)
-                    {
-                        if (currentCount != 0)
-                        {
-                            currentSegments.Add(new ResourceBindingSegment(
-                                currentDescriptor.Binding,
-                                currentCount,
-                                currentDescriptor.Type,
-                                currentDescriptor.Stages,
-                                currentDescriptor.Count > 1));
-                        }
-
-                        currentDescriptor = descriptor;
-                        currentCount = descriptor.Count;
-                    }
-                    else
-                    {
-                        currentCount += descriptor.Count;
-                    }
-                }
-
-                if (currentCount != 0)
-                {
-                    currentSegments.Add(new ResourceBindingSegment(
-                        currentDescriptor.Binding,
-                        currentCount,
-                        currentDescriptor.Type,
-                        currentDescriptor.Stages,
-                        currentDescriptor.Count > 1));
-                }
-
-                segments[setIndex] = currentSegments.ToArray();
-            }
-
-            return segments;
-        }
-
         private static (ResourceBindingSegment[][], int[], int[]) BuildBindingSegments(ReadOnlyCollection<ResourceUsageCollection> setUsages)
         {
             ResourceBindingSegment[][] segments = new ResourceBindingSegment[setUsages.Count][];