Skip to content

Commit

Permalink
wgsl: Revert changes to round execution tests
Browse files Browse the repository at this point in the history
These were landed on the wrong branch, and are causing tests to fail
to build, because some of the code that they depend on is implemented
in a PR that hasn't landed yet.
  • Loading branch information
zoddicus committed Oct 24, 2023
1 parent 8e7a995 commit 41ef8b0
Showing 1 changed file with 6 additions and 21 deletions.
27 changes: 6 additions & 21 deletions src/webgpu/shader/execution/expression/call/builtin/round.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,13 @@ Component-wise when T is a vector.

import { makeTestGroup } from '../../../../../../common/framework/test_group.js';
import { GPUTest } from '../../../../../gpu_test.js';
import { TypeF32, TypeF16, TypeAbstractFloat } from '../../../../../util/conversion.js';
import { TypeF32, TypeF16 } from '../../../../../util/conversion.js';
import { FP } from '../../../../../util/floating_point.js';
import { fullF32Range, fullF16Range, fullF64Range } from '../../../../../util/math.js';
import { fullF32Range, fullF16Range } from '../../../../../util/math.js';
import { makeCaseCache } from '../../case_cache.js';
import { allInputSources, onlyConstInputSource, run } from '../../expression.js';
import { allInputSources, run } from '../../expression.js';

import { abstractBuiltin, builtin } from './builtin.js';
import { builtin } from './builtin.js';

export const g = makeTestGroup(GPUTest);

Expand All @@ -43,30 +43,15 @@ export const d = makeCaseCache('round', {
FP.f16.roundInterval
);
},
abstract: () => {
return FP.abstract.generateScalarToIntervalCases(
[
0x8000_0000_0000_0000, // https://github.com/gpuweb/cts/issues/2766
...fullF64Range(),
],
'unfiltered',
FP.abstract.roundInterval
);
},
});

g.test('abstract_float')
.specURL('https://www.w3.org/TR/WGSL/#float-builtin-functions')
.desc(`abstract float tests`)
.params(u =>
u
.combine('inputSource', onlyConstInputSource)
.combine('vectorize', [undefined, 2, 3, 4] as const)
u.combine('inputSource', allInputSources).combine('vectorize', [undefined, 2, 3, 4] as const)
)
.fn(async t => {
const cases = await d.get('abstract');
await run(t, abstractBuiltin('round'), [TypeAbstractFloat], TypeAbstractFloat, t.params, cases);
});
.unimplemented();

g.test('f32')
.specURL('https://www.w3.org/TR/WGSL/#float-builtin-functions')
Expand Down

0 comments on commit 41ef8b0

Please sign in to comment.