Respond to PR feedback.

This commit is contained in:
Erin Maus
2026-07-24 08:06:52 -04:00
parent efa18664a0
commit d8430c39df
5 changed files with 3 additions and 10 deletions
-1
View File
@@ -2990,7 +2990,6 @@ STRINGMAP_CLASS_BEGIN(Graphics, Graphics::Feature, Graphics::FEATURE_MAX_ENUM, f
{ "indirectdraw", Graphics::FEATURE_INDIRECT_DRAW },
{ "vertexwrite", Graphics::FEATURE_VERTEX_WRITE },
{ "pixelwrite", Graphics::FEATURE_PIXEL_WRITE },
{ "imageatomics", Graphics::FEATURE_IMAGE_ATOMICS },
}
STRINGMAP_CLASS_END(Graphics, Graphics::Feature, Graphics::FEATURE_MAX_ENUM, feature)
-1
View File
@@ -165,7 +165,6 @@ public:
FEATURE_INDIRECT_DRAW,
FEATURE_VERTEX_WRITE,
FEATURE_PIXEL_WRITE,
FEATURE_IMAGE_ATOMICS,
FEATURE_MAX_ENUM
};
+1 -4
View File
@@ -2263,11 +2263,8 @@ void Graphics::initCapabilities()
capabilities.features[FEATURE_VERTEX_WRITE] = false;
capabilities.features[FEATURE_PIXEL_WRITE] = false;
}
// Not supported until Metal 3
capabilities.features[FEATURE_IMAGE_ATOMICS] = false;
static_assert(FEATURE_MAX_ENUM == 16, "Graphics::initCapabilities must be updated when adding a new graphics feature!");
static_assert(FEATURE_MAX_ENUM == 15, "Graphics::initCapabilities must be updated when adding a new graphics feature!");
// https://developer.apple.com/metal/Metal-Feature-Set-Tables.pdf
capabilities.limits[LIMIT_POINT_SIZE] = 511;
+1 -2
View File
@@ -1634,8 +1634,7 @@ void Graphics::initCapabilities()
capabilities.features[FEATURE_INDIRECT_DRAW] = capabilities.features[FEATURE_GLSL4];
capabilities.features[FEATURE_VERTEX_WRITE] = capabilities.features[FEATURE_GLSL4];
capabilities.features[FEATURE_PIXEL_WRITE] = capabilities.features[FEATURE_GLSL4];
capabilities.features[FEATURE_IMAGE_ATOMICS] = capabilities.features[FEATURE_GLSL4];
static_assert(FEATURE_MAX_ENUM == 16, "Graphics::initCapabilities must be updated when adding a new graphics feature!");
static_assert(FEATURE_MAX_ENUM == 15, "Graphics::initCapabilities must be updated when adding a new graphics feature!");
capabilities.limits[LIMIT_POINT_SIZE] = gl.getMaxPointSize();
capabilities.limits[LIMIT_TEXTURE_SIZE] = gl.getMax2DTextureSize();
+1 -2
View File
@@ -838,8 +838,7 @@ void Graphics::initCapabilities()
capabilities.features[FEATURE_INDIRECT_DRAW] = true;
capabilities.features[FEATURE_VERTEX_WRITE] = features.vertexPipelineStoresAndAtomics;
capabilities.features[FEATURE_PIXEL_WRITE] = features.fragmentStoresAndAtomics;
capabilities.features[FEATURE_IMAGE_ATOMICS] = true;
static_assert(FEATURE_MAX_ENUM == 16, "Graphics::initCapabilities must be updated when adding a new graphics feature!");
static_assert(FEATURE_MAX_ENUM == 15, "Graphics::initCapabilities must be updated when adding a new graphics feature!");
VkPhysicalDeviceProperties properties;
vkGetPhysicalDeviceProperties(physicalDevice, &properties);