Skip to content

Commit

Permalink
Changes for review
Browse files Browse the repository at this point in the history
* remove device selection for subgroup_uniformity
  • Loading branch information
alan-baker committed Dec 3, 2024
1 parent 97fc4f6 commit dbb53eb
Showing 1 changed file with 0 additions and 15 deletions.
15 changes: 0 additions & 15 deletions src/webgpu/shader/validation/parse/diagnostic.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -80,11 +80,6 @@ g.test('valid_params')
.combine('rule', kSpecDiagnosticRules)
.combine('type', kDiagnosticTypes)
)
.beforeAllSubcases(t => {
if (t.params.rule === 'subgroup_uniformity') {
t.selectDeviceOrSkipTestCase('subgroups' as GPUFeatureName);
}
})
.fn(t => {
const diag = generateDiagnostic(t.params.type, t.params.severity, t.params.rule);
let code = ``;
Expand Down Expand Up @@ -135,11 +130,6 @@ g.test('valid_locations')
.combine('location', keysOf(kValidLocations))
.combine('rule', kSpecDiagnosticRules)
)
.beforeAllSubcases(t => {
if (t.params.rule === 'subgroup_uniformity') {
t.selectDeviceOrSkipTestCase('subgroups' as GPUFeatureName);
}
})
.fn(t => {
const diag = generateDiagnostic(t.params.type, 'info', t.params.rule);
const code = kValidLocations[t.params.location](diag);
Expand All @@ -164,11 +154,6 @@ g.test('invalid_locations')
.combine('location', keysOf(kInvalidLocations))
.combine('rule', kSpecDiagnosticRules)
)
.beforeAllSubcases(t => {
if (t.params.rule === 'subgroup_uniformity') {
t.selectDeviceOrSkipTestCase('subgroups' as GPUFeatureName);
}
})
.fn(t => {
const diag = generateDiagnostic(t.params.type, 'info', t.params.rule);
t.expectCompileResult(true, kInvalidLocations[t.params.location](''));
Expand Down

0 comments on commit dbb53eb

Please sign in to comment.