subgroup ops refactor, matches glsl more closely

Firestar99 · Firestar99 · commit 63a03d4a6678 · 2024-10-12T13:25:34.000+02:00
diff --git a/crates/spirv-std/src/arch/subgroup.rs b/crates/spirv-std/src/arch/subgroup.rs
diff --git a/tests/ui/arch/subgroup/subgroup_ballot.stderr b/tests/ui/arch/subgroup/subgroup_ballot.stderr
@@ -1,9 +1,9 @@
 %1 = OpFunction  %2  None %3
 %4 = OpFunctionParameter  %2
 %5 = OpLabel
-OpLine %6 379 8
+OpLine %6 380 8
 %7 = OpGroupNonUniformBallot  %8  %9 %4
-OpLine %6 415 8
+OpLine %6 416 8
 %10 = OpGroupNonUniformInverseBallot  %2  %9 %7
 OpNoLine
 OpReturnValue %10
diff --git a/tests/ui/arch/subgroup/subgroup_ballot_bit_count.rs b/tests/ui/arch/subgroup/subgroup_ballot_bit_count.rs
@@ -6,7 +6,7 @@ use spirv_std::arch::{GroupOperation, SubgroupMask};
 use spirv_std::spirv;
 
 unsafe fn subgroup_ballot_bit_count(ballot: SubgroupMask) -> u32 {
-    spirv_std::arch::subgroup_ballot_bit_count::<{ GroupOperation::Reduce as u32 }>(ballot)
+    spirv_std::arch::subgroup_ballot_bit_count(ballot)
 }
 
 #[spirv(compute(threads(1, 1, 1)))]
diff --git a/tests/ui/arch/subgroup/subgroup_ballot_bit_count.stderr b/tests/ui/arch/subgroup/subgroup_ballot_bit_count.stderr
@@ -1,7 +1,7 @@
 %1 = OpFunction  %2  None %3
 %4 = OpFunctionParameter  %5
 %6 = OpLabel
-OpLine %7 491 8
+OpLine %7 512 0
 %8 = OpGroupNonUniformBallotBitCount  %2  %9 Reduce %4
 OpNoLine
 OpReturnValue %8
diff --git a/tests/ui/arch/subgroup/subgroup_broadcast_first.stderr b/tests/ui/arch/subgroup/subgroup_broadcast_first.stderr
@@ -1,7 +1,7 @@
 %1 = OpFunction  %2  None %3
 %4 = OpFunctionParameter  %2
 %5 = OpLabel
-OpLine %6 346 8
+OpLine %6 347 8
 %7 = OpGroupNonUniformBroadcastFirst  %2  %8 %4
 OpNoLine
 OpReturnValue %7
diff --git a/tests/ui/arch/subgroup/subgroup_elect.stderr b/tests/ui/arch/subgroup/subgroup_elect.stderr
@@ -1,6 +1,6 @@
 %1 = OpFunction  %2  None %3
 %4 = OpLabel
-OpLine %5 181 8
+OpLine %5 182 8
 %6 = OpGroupNonUniformElect  %2  %7
 OpNoLine
 OpReturnValue %6
diff --git a/tests/ui/arch/subgroup/subgroup_i_add_clustered.rs b/tests/ui/arch/subgroup/subgroup_i_add_clustered.rs
@@ -7,7 +7,7 @@ use spirv_std::arch::{GroupOperation, SubgroupMask};
 use spirv_std::spirv;
 
 unsafe fn subgroup_i_add_clustered(value: u32) -> u32 {
-    spirv_std::arch::subgroup_i_add_clustered::<8, _>(value)
+    spirv_std::arch::subgroup_clustered_i_add::<8, _>(value)
 }
 
 #[spirv(compute(threads(32, 1, 1)))]
diff --git a/tests/ui/arch/subgroup/subgroup_i_add_clustered.stderr b/tests/ui/arch/subgroup/subgroup_i_add_clustered.stderr
@@ -1,7 +1,7 @@
 %1 = OpFunction  %2  None %3
 %4 = OpFunctionParameter  %2
 %5 = OpLabel
-OpLine %6 782 8
+OpLine %6 810 0
 %7 = OpGroupNonUniformIAdd  %2  %8 ClusteredReduce %4 %9
 OpNoLine
 OpReturnValue %7
diff --git a/tests/ui/arch/subgroup/subgroup_i_add_exclusive_scan.rs b/tests/ui/arch/subgroup/subgroup_i_add_exclusive_scan.rs
@@ -7,7 +7,7 @@ use spirv_std::arch::{GroupOperation, SubgroupMask};
 use spirv_std::spirv;
 
 unsafe fn subgroup_i_add_exclusive_scan(value: u32) -> u32 {
-    spirv_std::arch::subgroup_i_add::<{ GroupOperation::ExclusiveScan as u32 }, _>(value)
+    spirv_std::arch::subgroup_exclusive_i_add(value)
 }
 
 #[spirv(compute(threads(32, 1, 1)))]
diff --git a/tests/ui/arch/subgroup/subgroup_i_add_exclusive_scan.stderr b/tests/ui/arch/subgroup/subgroup_i_add_exclusive_scan.stderr
@@ -1,7 +1,7 @@
 %1 = OpFunction  %2  None %3
 %4 = OpFunctionParameter  %2
 %5 = OpLabel
-OpLine %6 741 8
+OpLine %6 797 0
 %7 = OpGroupNonUniformIAdd  %2  %8 ExclusiveScan %4
 OpNoLine
 OpReturnValue %7
diff --git a/tests/ui/arch/subgroup/subgroup_i_add_inclusive_scan.rs b/tests/ui/arch/subgroup/subgroup_i_add_inclusive_scan.rs
@@ -7,7 +7,7 @@ use spirv_std::arch::{GroupOperation, SubgroupMask};
 use spirv_std::spirv;
 
 unsafe fn subgroup_i_add_inclusive_scan(value: u32) -> u32 {
-    spirv_std::arch::subgroup_i_add::<{ GroupOperation::InclusiveScan as u32 }, _>(value)
+    spirv_std::arch::subgroup_inclusive_i_add(value)
 }
 
 #[spirv(compute(threads(32, 1, 1)))]
diff --git a/tests/ui/arch/subgroup/subgroup_i_add_inclusive_scan.stderr b/tests/ui/arch/subgroup/subgroup_i_add_inclusive_scan.stderr
@@ -1,7 +1,7 @@
 %1 = OpFunction  %2  None %3
 %4 = OpFunctionParameter  %2
 %5 = OpLabel
-OpLine %6 741 8
+OpLine %6 797 0
 %7 = OpGroupNonUniformIAdd  %2  %8 InclusiveScan %4
 OpNoLine
 OpReturnValue %7
diff --git a/tests/ui/arch/subgroup/subgroup_i_add_reduce.rs b/tests/ui/arch/subgroup/subgroup_i_add_reduce.rs
@@ -7,7 +7,7 @@ use spirv_std::arch::{GroupOperation, SubgroupMask};
 use spirv_std::spirv;
 
 unsafe fn subgroup_i_add_reduce(value: u32) -> u32 {
-    spirv_std::arch::subgroup_i_add::<{ GroupOperation::Reduce as u32 }, _>(value)
+    spirv_std::arch::subgroup_i_add(value)
 }
 
 #[spirv(compute(threads(32, 1, 1)))]
diff --git a/tests/ui/arch/subgroup/subgroup_i_add_reduce.stderr b/tests/ui/arch/subgroup/subgroup_i_add_reduce.stderr
@@ -1,7 +1,7 @@
 %1 = OpFunction  %2  None %3
 %4 = OpFunctionParameter  %2
 %5 = OpLabel
-OpLine %6 741 8
+OpLine %6 797 0
 %7 = OpGroupNonUniformIAdd  %2  %8 Reduce %4
 OpNoLine
 OpReturnValue %7

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ use spirv_std::arch::{GroupOperation, SubgroupMask};`
`6`	`6`	`use spirv_std::spirv;`
`7`	`7`
`8`	`8`	`unsafe fn subgroup_ballot_bit_count(ballot: SubgroupMask) -> u32 {`
`9`		`- spirv_std::arch::subgroup_ballot_bit_count::<{ GroupOperation::Reduce as u32 }>(ballot)`
	`9`	`+ spirv_std::arch::subgroup_ballot_bit_count(ballot)`
`10`	`10`	`}`
`11`	`11`
`12`	`12`	`#[spirv(compute(threads(1, 1, 1)))]`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ use spirv_std::arch::{GroupOperation, SubgroupMask};`
`7`	`7`	`use spirv_std::spirv;`
`8`	`8`
`9`	`9`	`unsafe fn subgroup_i_add_clustered(value: u32) -> u32 {`
`10`		`- spirv_std::arch::subgroup_i_add_clustered::<8, _>(value)`
	`10`	`+ spirv_std::arch::subgroup_clustered_i_add::<8, _>(value)`
`11`	`11`	`}`
`12`	`12`
`13`	`13`	`#[spirv(compute(threads(32, 1, 1)))]`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ use spirv_std::arch::{GroupOperation, SubgroupMask};`
`7`	`7`	`use spirv_std::spirv;`
`8`	`8`
`9`	`9`	`unsafe fn subgroup_i_add_exclusive_scan(value: u32) -> u32 {`
`10`		`- spirv_std::arch::subgroup_i_add::<{ GroupOperation::ExclusiveScan as u32 }, _>(value)`
	`10`	`+ spirv_std::arch::subgroup_exclusive_i_add(value)`
`11`	`11`	`}`
`12`	`12`
`13`	`13`	`#[spirv(compute(threads(32, 1, 1)))]`