Apply suggestions from code review

Co-authored-by: Teodor Tanasoaia <28601907+teoxoy@users.noreply.github.com>

Author: andyleiserson

wgpu-core/src/limits.rs

@@ -307,10 +307,10 @@ const UPLEVEL: Bucket = Bucket {
         max_inter_stage_shader_variables: 28,
         // use default max_sampled_textures_per_shader_stage
         // use default max_samplers_per_shader_stage
-        max_storage_buffer_binding_size: 1 << 27, // 128 MB
+        // use default max_storage_buffer_binding_size
         // wgpu does not implement max_storage_buffers_in_fragment_stage: 8,
         // wgpu does not implement max_storage_buffers_in_vertex_stage: 8,
-        max_storage_buffers_per_shader_stage: 8,
+        // use default max_storage_buffers_per_shader_stage
         // wgpu does not implement max_storage_textures_in_fragment_stage: 8,
         // wgpu does not implement max_storage_textures_in_vertex_stage: 8,
         max_storage_textures_per_shader_stage: 8,
@@ -346,7 +346,7 @@ const UPLEVEL: Bucket = Bucket {
         .union(Features::BGRA8UNORM_STORAGE)
         .union(Features::FLOAT32_FILTERABLE)
         // FLOAT32_BLENDABLE not implemented in wgpu dx12 backend; https://github.com/gfx-rs/wgpu/issues/6555
-        // CLIP_DISTANCES not implemented in wgpu Metal backend; https://github.com/gfx-rs/wgpu/issues/6236
+        // CLIP_DISTANCES not implemented in wgpu dx12 or Metal backend; https://github.com/gfx-rs/wgpu/issues/6236
         .union(Features::DUAL_SOURCE_BLENDING)
         // TIER1/TIER2 not implemented in wgpu; https://github.com/gfx-rs/wgpu/issues/8122
         .union(Features::PRIMITIVE_INDEX)
@@ -396,10 +396,7 @@ const BUCKET_A2: Bucket = Bucket {
         subgroup_max_size: 64,
         ..UPLEVEL.info
     },
-    features: UPLEVEL
-        .features
-        .union(Features::SHADER_F16)
-        .union(Features::CLIP_DISTANCES),
+    features: UPLEVEL.features.union(Features::SHADER_F16),
 };
 
 // e.g. Intel Arc, UHD 600 Series, Iris Xe
@@ -417,10 +414,7 @@ const BUCKET_I1: Bucket = Bucket {
         subgroup_max_size: 32,
         ..UPLEVEL.info
     },
-    features: UPLEVEL
-        .features
-        .union(Features::SHADER_F16)
-        .union(Features::CLIP_DISTANCES),
+    features: UPLEVEL.features.union(Features::SHADER_F16),
 };
 
 // e.g. GeForce GTX 1650, GeForce RTX 20, 30, 40, 50 Series
@@ -440,10 +434,7 @@ const BUCKET_N1: Bucket = Bucket {
         subgroup_max_size: 32,
         ..UPLEVEL.info
     },
-    features: UPLEVEL
-        .features
-        .union(Features::SHADER_F16)
-        .union(Features::CLIP_DISTANCES),
+    features: UPLEVEL.features.union(Features::SHADER_F16),
 };
 
 // e.g. Radeon RX 6000, 7000, 9000 Series
@@ -462,10 +453,7 @@ const BUCKET_A1: Bucket = Bucket {
         subgroup_max_size: 64,
         ..UPLEVEL.info
     },
-    features: UPLEVEL
-        .features
-        .union(Features::SHADER_F16)
-        .union(Features::CLIP_DISTANCES),
+    features: UPLEVEL.features.union(Features::SHADER_F16),
 };
 
 // e.g. GeForce GTX 1050, Radeon WX 5100
@@ -485,7 +473,7 @@ const BUCKET_NO_F16: Bucket = Bucket {
         subgroup_max_size: 64,
         ..UPLEVEL.info
     },
-    features: UPLEVEL.features.union(Features::CLIP_DISTANCES),
+    features: UPLEVEL.features,
 };
 
 const BUCKET_LLVMPIPE: Bucket = Bucket {
@@ -523,10 +511,7 @@ const BUCKET_WARP: Bucket = Bucket {
         subgroup_min_size: 4,
         subgroup_max_size: 128,
     },
-    features: UPLEVEL
-        .features
-        .union(Features::SHADER_F16)
-        .union(Features::CLIP_DISTANCES),
+    features: UPLEVEL.features.union(Features::SHADER_F16),
 };
 
 // WebGPU default limits, not a fallback adapter