gpuweb / cts

WebGPU Conformance Test Suite
https://gpuweb.github.io/cts/
BSD 3-Clause "New" or "Revised" License
128 stars 80 forks source link

wgsl: Migrate AF 'fract' to use look up table #3684

Closed zoddicus closed 5 months ago

zoddicus commented 5 months ago

This updates the fract tests to be compliant with a recent change to the spec, https://github.com/gpuweb/gpuweb/pull/4541.

This also includes some refactoring of the existing generation code for better reuse, removing some spurious zeros in the output, and avoiding potentially ambigious fractional values, by changing them to hex values.

Fixes https://github.com/gpuweb/cts/issues/3672


Requirements for PR author:

Requirements for reviewer sign-off:

When landing this PR, be sure to make any necessary issue status updates.

zoddicus commented 5 months ago

Depends on the AF subtraction PR, so the new code is in https://github.com/gpuweb/cts/pull/3684/commits/1e7027b49a5360bffd597d26b6dc647eb719e7e8

zoddicus commented 5 months ago

@dneto0 Can I get a review on the dependeny PR, https://github.com/gpuweb/cts/pull/3667

zoddicus commented 5 months ago

@dneto0 Can I get a review on the dependeny PR, #3667

Dan reviewed