WIP: refactor: use plural "clip distances" for cross-platform layers

ErichDonGubler · ErichDonGubler · commit 2d8e6819e413 · 2026-02-02T16:12:51.000-05:00
diff --git a/naga-cli/src/bin/naga.rs b/naga-cli/src/bin/naga.rs
@@ -603,7 +603,7 @@ fn run() -> anyhow::Result<()> {
             .fold(naga::valid::Capabilities::all(), |caps, path| {
                 use naga::valid::Capabilities as C;
                 let missing = match Path::new(path).extension().and_then(|ex| ex.to_str()) {
-                    Some("wgsl") => C::CLIP_DISTANCE | C::CULL_DISTANCE,
+                    Some("wgsl") => C::CLIP_DISTANCES | C::CULL_DISTANCE,
                     Some("metal") => C::CULL_DISTANCE,
                     Some("hlsl") => C::empty(),
                     _ => C::TEXTURE_EXTERNAL,
diff --git a/naga/src/back/glsl/conv.rs b/naga/src/back/glsl/conv.rs
@@ -76,7 +76,7 @@ pub(in crate::back::glsl) const fn glsl_built_in(
         // vertex
         Bi::BaseInstance => "uint(gl_BaseInstance)",
         Bi::BaseVertex => "uint(gl_BaseVertex)",
-        Bi::ClipDistance => "gl_ClipDistance",
+        Bi::ClipDistances => "gl_ClipDistance",
         Bi::CullDistance => "gl_CullDistance",
         Bi::InstanceIndex => {
             if options.draw_parameters {
diff --git a/naga/src/back/glsl/features.rs b/naga/src/back/glsl/features.rs
@@ -604,7 +604,7 @@ impl<W> Writer<'_, W> {
         } else if let Some(binding) = binding {
             match *binding {
                 Binding::BuiltIn(built_in) => match built_in {
-                    crate::BuiltIn::ClipDistance => self.features.request(Features::CLIP_DISTANCE),
+                    crate::BuiltIn::ClipDistances => self.features.request(Features::CLIP_DISTANCE),
                     crate::BuiltIn::CullDistance => self.features.request(Features::CULL_DISTANCE),
                     crate::BuiltIn::SampleIndex => {
                         self.features.request(Features::SAMPLE_VARIABLES)
diff --git a/naga/src/back/glsl/writer.rs b/naga/src/back/glsl/writer.rs
@@ -1072,7 +1072,7 @@ impl<'a, W: Write> Writer<'a, W> {
                             }
                         }
                     }
-                    crate::BuiltIn::ClipDistance => {
+                    crate::BuiltIn::ClipDistances => {
                         // Re-declare `gl_ClipDistance` with number of clip planes.
                         let TypeInner::Array { size, .. } = self.module.types[ty].inner else {
                             unreachable!();
diff --git a/naga/src/back/hlsl/conv.rs b/naga/src/back/hlsl/conv.rs
@@ -153,7 +153,7 @@ impl crate::BuiltIn {
         Ok(match self {
             Self::Position { .. } => "SV_Position",
             // vertex
-            Self::ClipDistance => "SV_ClipDistance",
+            Self::ClipDistances => "SV_ClipDistance",
             Self::CullDistance => "SV_CullDistance",
             Self::InstanceIndex => "SV_InstanceID",
             Self::VertexIndex => "SV_VertexID",
diff --git a/naga/src/back/msl/mod.rs b/naga/src/back/msl/mod.rs
@@ -686,7 +686,7 @@ impl ResolvedBinding {
                     // vertex
                     Bi::BaseInstance => "base_instance",
                     Bi::BaseVertex => "base_vertex",
-                    Bi::ClipDistance => "clip_distance",
+                    Bi::ClipDistances => "clip_distance",
                     Bi::InstanceIndex => "instance_id",
                     Bi::PointSize => "point_size",
                     Bi::VertexIndex => "vertex_id",
diff --git a/naga/src/back/spv/writer.rs b/naga/src/back/spv/writer.rs
@@ -3060,9 +3060,9 @@ impl Writer {
                     // vertex
                     Bi::BaseInstance => BuiltIn::BaseInstance,
                     Bi::BaseVertex => BuiltIn::BaseVertex,
-                    Bi::ClipDistance => {
+                    Bi::ClipDistances => {
                         self.require_any(
-                            "`clip_distance` built-in",
+                            "`clip_distances` built-in",
                             &[spirv::Capability::ClipDistance],
                         )?;
                         BuiltIn::ClipDistance
diff --git a/naga/src/back/wgsl/writer.rs b/naga/src/back/wgsl/writer.rs
@@ -300,7 +300,7 @@ impl<W: Write> Writer<W> {
                             } => {
                                 needs_dual_source_blending = true;
                             }
-                            crate::Binding::BuiltIn(crate::BuiltIn::ClipDistance) => {
+                            crate::Binding::BuiltIn(crate::BuiltIn::ClipDistances) => {
                                 needs_clip_distances = true;
                             }
                             crate::Binding::Location {
diff --git a/naga/src/common/wgsl/to_wgsl.rs b/naga/src/common/wgsl/to_wgsl.rs
@@ -165,7 +165,7 @@ impl TryToWgsl for crate::BuiltIn {
             Bi::ViewIndex => "view_index",
             Bi::InstanceIndex => "instance_index",
             Bi::VertexIndex => "vertex_index",
-            Bi::ClipDistance => "clip_distances",
+            Bi::ClipDistances => "clip_distances",
             Bi::FragDepth => "frag_depth",
             Bi::FrontFacing => "front_facing",
             Bi::PrimitiveIndex => "primitive_index",
diff --git a/naga/src/front/glsl/variables.rs b/naga/src/front/glsl/variables.rs
@@ -188,7 +188,7 @@ impl Frontend {
                         stride: 4,
                     },
                     builtin: match name {
-                        "gl_ClipDistance" => BuiltIn::ClipDistance,
+                        "gl_ClipDistance" => BuiltIn::ClipDistances,
                         "gl_CullDistance" => BuiltIn::CullDistance,
                         _ => unreachable!(),
                     },
diff --git a/naga/src/front/spv/convert.rs b/naga/src/front/spv/convert.rs
@@ -136,7 +136,7 @@ pub(super) fn map_builtin(word: spirv::Word, invariant: bool) -> Result<crate::B
         // vertex
         Some(Bi::BaseInstance) => crate::BuiltIn::BaseInstance,
         Some(Bi::BaseVertex) => crate::BuiltIn::BaseVertex,
-        Some(Bi::ClipDistance) => crate::BuiltIn::ClipDistance,
+        Some(Bi::ClipDistance) => crate::BuiltIn::ClipDistances,
         Some(Bi::CullDistance) => crate::BuiltIn::CullDistance,
         Some(Bi::InstanceIndex) => crate::BuiltIn::InstanceIndex,
         Some(Bi::PointSize) => crate::BuiltIn::PointSize,
diff --git a/naga/src/front/wgsl/parse/conv.rs b/naga/src/front/wgsl/parse/conv.rs
@@ -105,7 +105,7 @@ pub fn map_built_in(
         "vertex_index" => crate::BuiltIn::VertexIndex,
         "instance_index" => crate::BuiltIn::InstanceIndex,
         "view_index" => crate::BuiltIn::ViewIndex,
-        "clip_distances" => crate::BuiltIn::ClipDistance,
+        "clip_distances" => crate::BuiltIn::ClipDistances,
         // fragment
         "front_facing" => crate::BuiltIn::FrontFacing,
         "frag_depth" => crate::BuiltIn::FragDepth,
@@ -153,7 +153,7 @@ pub fn map_built_in(
         _ => return Err(Box::new(Error::UnknownBuiltin(span))),
     };
     match built_in {
-        crate::BuiltIn::ClipDistance => {
+        crate::BuiltIn::ClipDistances => {
             if !enable_extensions.contains(ImplementedEnableExtension::ClipDistances) {
                 return Err(Box::new(Error::EnableExtensionNotEnabled {
                     span,
diff --git a/naga/src/ir/mod.rs b/naga/src/ir/mod.rs
@@ -411,7 +411,7 @@ pub enum BuiltIn {
     /// Read in vertex shaders
     BaseVertex,
     /// Written in vertex & mesh shaders
-    ClipDistance,
+    ClipDistances,
     /// Written in vertex & mesh shaders
     CullDistance,
     /// Read in vertex, any- and closest-hit shaders
diff --git a/naga/src/valid/interface.rs b/naga/src/valid/interface.rs
@@ -241,7 +241,7 @@ impl VaryingContext<'_> {
                 self.built_ins.insert(canonical);
 
                 let required = match built_in {
-                    Bi::ClipDistance => Capabilities::CLIP_DISTANCE,
+                    Bi::ClipDistances => Capabilities::CLIP_DISTANCES,
                     Bi::CullDistance => Capabilities::CULL_DISTANCE,
                     // Primitive index is allowed w/o any other extensions in any- and closest-hit shaders
                     Bi::PrimitiveIndex if !matches!(ep.stage, St::AnyHit | St::ClosestHit) => {
@@ -286,7 +286,7 @@ impl VaryingContext<'_> {
                             && !self.output,
                         *ty_inner == Ti::Scalar(crate::Scalar::U32),
                     ),
-                    Bi::ClipDistance | Bi::CullDistance => (
+                    Bi::ClipDistances | Bi::CullDistance => (
                         (self.stage == St::Vertex || self.stage == St::Mesh) && self.output,
                         match *ty_inner {
                             Ti::Array { base, size, .. } => {
diff --git a/naga/src/valid/mod.rs b/naga/src/valid/mod.rs
@@ -103,10 +103,10 @@ bitflags::bitflags! {
         const STORAGE_TEXTURE_BINDING_ARRAY = 1 << 5;
         /// Support for binding arrays of storage buffers.
         const STORAGE_BUFFER_BINDING_ARRAY = 1 << 6;
-        /// Support for [`BuiltIn::ClipDistance`].
+        /// Support for [`BuiltIn::ClipDistances`].
         ///
-        /// [`BuiltIn::ClipDistance`]: crate::BuiltIn::ClipDistance
-        const CLIP_DISTANCE = 1 << 7;
+        /// [`BuiltIn::ClipDistances`]: crate::BuiltIn::ClipDistances
+        const CLIP_DISTANCES = 1 << 7;
         /// Support for [`BuiltIn::CullDistance`].
         ///
         /// [`BuiltIn::CullDistance`]: crate::BuiltIn::CullDistance
@@ -222,7 +222,7 @@ impl Capabilities {
             Self::DUAL_SOURCE_BLENDING => Some(Ext::DualSourceBlending),
             // NOTE: `SHADER_FLOAT16_IN_FLOAT32` _does not_ require the `f16` extension
             Self::SHADER_FLOAT16 => Some(Ext::F16),
-            Self::CLIP_DISTANCE => Some(Ext::ClipDistances),
+            Self::CLIP_DISTANCES => Some(Ext::ClipDistances),
             Self::MESH_SHADER => Some(Ext::WgpuMeshShader),
             Self::RAY_QUERY => Some(Ext::WgpuRayQuery),
             Self::RAY_HIT_VERTEX_POSITION => Some(Ext::WgpuRayQueryVertexReturn),
diff --git a/naga/tests/in/wgsl/clip-distances.toml b/naga/tests/in/wgsl/clip-distances.toml
@@ -1,4 +1,4 @@
-capabilities = "CLIP_DISTANCE"
+capabilities = "CLIP_DISTANCES"
 targets = "SPIRV | GLSL | WGSL"
 
 [glsl]
diff --git a/naga/tests/naga/wgsl_errors.rs b/naga/tests/naga/wgsl_errors.rs
@@ -4113,7 +4113,7 @@ fn subgroup_invalid_broadcast() {
 fn invalid_clip_distances() {
     // Missing capability or enable directive
     check_extension_validation! {
-        Capabilities::CLIP_DISTANCE,
+        Capabilities::CLIP_DISTANCES,
         r#"
             @vertex
             fn vs_main() -> @builtin(clip_distances) array<f32, 8> {
@@ -4133,7 +4133,7 @@ fn invalid_clip_distances() {
         Err(naga::valid::ValidationError::EntryPoint {
             stage: naga::ShaderStage::Vertex,
             source: naga::valid::EntryPointError::Result(
-                naga::valid::VaryingError::UnsupportedCapability(Capabilities::CLIP_DISTANCE)
+                naga::valid::VaryingError::UnsupportedCapability(Capabilities::CLIP_DISTANCES)
             ),
             ..
         })
@@ -4157,11 +4157,11 @@ fn invalid_clip_distances() {
         Err(naga::valid::ValidationError::EntryPoint {
             stage: naga::ShaderStage::Vertex,
             source: naga::valid::EntryPointError::Result(
-                naga::valid::VaryingError::InvalidBuiltInType(naga::ir::BuiltIn::ClipDistance, _)
+                naga::valid::VaryingError::InvalidBuiltInType(naga::ir::BuiltIn::ClipDistances, _)
             ),
             ..
         }),
-        naga::valid::Capabilities::CLIP_DISTANCE
+        naga::valid::Capabilities::CLIP_DISTANCES
     }
 }
 
diff --git a/wgpu-core/src/device/mod.rs b/wgpu-core/src/device/mod.rs
@@ -487,7 +487,7 @@ pub fn features_to_naga_capabilities(
         features.contains(wgt::Features::DUAL_SOURCE_BLENDING),
     );
     caps.set(
-        Caps::CLIP_DISTANCE,
+        Caps::CLIP_DISTANCES,
         features.contains(wgt::Features::CLIP_DISTANCES),
     );
     caps.set(
diff --git a/wgpu-core/src/validation.rs b/wgpu-core/src/validation.rs
@@ -161,7 +161,7 @@ enum BuiltIn {
     ViewIndex,
     BaseInstance,
     BaseVertex,
-    ClipDistance { array_size: u32 },
+    ClipDistances { array_size: u32 },
     CullDistance,
     InstanceIndex,
     PointSize,
@@ -215,7 +215,7 @@ impl BuiltIn {
             Self::ViewIndex => naga::BuiltIn::ViewIndex,
             Self::BaseInstance => naga::BuiltIn::BaseInstance,
             Self::BaseVertex => naga::BuiltIn::BaseVertex,
-            Self::ClipDistance { .. } => naga::BuiltIn::ClipDistance,
+            Self::ClipDistances { .. } => naga::BuiltIn::ClipDistances,
             Self::CullDistance => naga::BuiltIn::CullDistance,
             Self::InstanceIndex => naga::BuiltIn::InstanceIndex,
             Self::PointSize => naga::BuiltIn::PointSize,
@@ -1188,7 +1188,7 @@ impl Interface {
             naga::TypeInner::Array { base, size, stride }
                 if matches!(
                     binding,
-                    Some(naga::Binding::BuiltIn(naga::BuiltIn::ClipDistance)),
+                    Some(naga::Binding::BuiltIn(naga::BuiltIn::ClipDistances)),
                 ) =>
             {
                 // NOTE: We should already have validated these in `naga`.
@@ -1209,7 +1209,7 @@ impl Interface {
                 };
                 let array_size = array_size.get();
 
-                list.push(Varying::BuiltIn(BuiltIn::ClipDistance { array_size }));
+                list.push(Varying::BuiltIn(BuiltIn::ClipDistances { array_size }));
                 return;
             }
             ref other => {
@@ -1244,7 +1244,7 @@ impl Interface {
                 naga::BuiltIn::ViewIndex => BuiltIn::ViewIndex,
                 naga::BuiltIn::BaseInstance => BuiltIn::BaseInstance,
                 naga::BuiltIn::BaseVertex => BuiltIn::BaseVertex,
-                naga::BuiltIn::ClipDistance => unreachable!(),
+                naga::BuiltIn::ClipDistances => unreachable!(),
                 naga::BuiltIn::CullDistance => BuiltIn::CullDistance,
                 naga::BuiltIn::InstanceIndex => BuiltIn::InstanceIndex,
                 naga::BuiltIn::PointSize => BuiltIn::PointSize,
@@ -1724,15 +1724,15 @@ impl Interface {
                 };
 
                 let clip_distance_deductions = entry_point.outputs.iter().filter_map(|output| {
-                    if let &Varying::BuiltIn(BuiltIn::ClipDistance { array_size }) = output {
-                        Some(MaxVertexShaderOutputDeduction::ClipDistance { array_size })
+                    if let &Varying::BuiltIn(BuiltIn::ClipDistances { array_size }) = output {
+                        Some(MaxVertexShaderOutputDeduction::ClipDistances { array_size })
                     } else {
                         None
                     }
                 });
                 debug_assert!(
                     clip_distance_deductions.clone().count() <= 1,
-                    "multiple `clip_distance` outputs found"
+                    "multiple `clip_distances` outputs found"
                 );
 
                 let deductions = point_list_deduction
diff --git a/wgpu-core/src/validation/shader_io_deductions.rs b/wgpu-core/src/validation/shader_io_deductions.rs
@@ -13,7 +13,7 @@ pub enum MaxVertexShaderOutputDeduction {
     /// [`wgt::PrimitiveTopology::PointList`].
     PointListPrimitiveTopology,
     /// When a clip distances array primitive is used in an output.
-    ClipDistance { array_size: u32 },
+    ClipDistances { array_size: u32 },
 }
 
 impl MaxVertexShaderOutputDeduction {
@@ -26,7 +26,7 @@ impl MaxVertexShaderOutputDeduction {
     pub fn for_variables(self) -> u32 {
         match self {
             Self::PointListPrimitiveTopology => 1,
-            Self::ClipDistance { array_size } => {
+            Self::ClipDistances { array_size } => {
                 Self::variables_from_clip_distance_slot(array_size)
             }
         }
@@ -35,7 +35,7 @@ impl MaxVertexShaderOutputDeduction {
     pub fn for_location(self) -> u32 {
         match self {
             Self::PointListPrimitiveTopology => 0,
-            Self::ClipDistance { array_size } => {
+            Self::ClipDistances { array_size } => {
                 Self::variables_from_clip_distance_slot(array_size)
             }
         }
@@ -84,7 +84,7 @@ impl MaxFragmentShaderInputDeduction {
             BuiltIn::ViewIndex => InterStageBuiltIn::ViewIndex,
             BuiltIn::BaseInstance
             | BuiltIn::BaseVertex
-            | BuiltIn::ClipDistance
+            | BuiltIn::ClipDistances
             | BuiltIn::CullDistance
             | BuiltIn::InstanceIndex
             | BuiltIn::PointSize

Original file line number	Diff line number	Diff line change
`@@ -1072,7 +1072,7 @@ impl<'a, W: Write> Writer<'a, W> {`
`1072`	`1072`	`}`
`1073`	`1073`	`}`
`1074`	`1074`	`}`
`1075`		`- crate::BuiltIn::ClipDistance => {`
	`1075`	`+ crate::BuiltIn::ClipDistances => {`
`1076`	`1076`	// Re-declare `gl_ClipDistance` with number of clip planes.
`1077`	`1077`	`let TypeInner::Array { size, .. } = self.module.types[ty].inner else {`
`1078`	`1078`	`unreachable!();`
Original file line number	Diff line number	Diff line change
`@@ -300,7 +300,7 @@ impl<W: Write> Writer<W> {`
`300`	`300`	`} => {`
`301`	`301`	`needs_dual_source_blending = true;`
`302`	`302`	`}`
`303`		`- crate::Binding::BuiltIn(crate::BuiltIn::ClipDistance) => {`
	`303`	`+ crate::Binding::BuiltIn(crate::BuiltIn::ClipDistances) => {`
`304`	`304`	`needs_clip_distances = true;`
`305`	`305`	`}`
`306`	`306`	`crate::Binding::Location {`