Ben Ashbaugh
Ben Ashbaugh
> Quick experiment (for `half` test) with below modification gave promising result: [...] This seems reasonable to me and appears to produce the expected amount of testing: ``` Function Name,...
This PR needs a new owner, so for now I've removed "focused review" and added "help wanted".
`fma` is failing also, if I take off `-w`: ``` 25: fma fp16 .... ERROR: fma: -0.500000 ulp error at {0x1.eacp+7, 0x1.3f4p+4, 0x1.c04p+14} ({0x5bab, 0x4cfd, 0x7701}): *0x1.068p+15 vs. 0x1.064p+15 fma...
> I'm also seeing a failure with `divide_cr`: It looks like the tests are checking for a correctly rounded result in this case, but it's not clear this is required....
We're still debugging a few unexpected failures on our side, also. Can we merge these changes but with half testing disabled by default, able to be enabled via command line...
I just created an `fp16-staging` branch, as we discussed. If this PR is retargeted at this branch I will merge it, also as we discussed.
Removing "focused review", since we are reviewing and merging changes into the staging branch now.
Closing as discussed in the August 27th teleconference.
A good initial step towards resolving this issue would be a test plan documenting what will be tested.
Discussed in the June 18th teleconference. This is an older PR that has been rebased and updated and is now ready for review.