-
Notifications
You must be signed in to change notification settings - Fork 130
[SYCL][libdevice] Tests for half value type cast utils. #1301
Conversation
This PR is test for intel/llvm#6930 |
Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 <[email protected]>
} | ||
|
||
{ | ||
std::initializer_list<uint64_t> input_vals = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this just a sanity check or detailed test?
The test is good as a sanity one.
For detailed test I would suggest to extend it by:
- all DP corner points: +/-zero, +/-max&min subnormals, +/-max&min normals, +/-infs, nans
- all HP expected output corner points: +/-zero, +/-max&min subnormals, +/-max&min normals
(some of them are already covered) -
- all possible exponent fields (with random mantissa)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, @akolesov-intel
This test is a lit test which is similar to sanity test. And I also added +0/-0, +/-max & min subnormals, +/- max & min normals, +/-infs
Thanks very much.
Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 <[email protected]>
…rted on CPU device Signed-off-by: jinge90 <[email protected]>
Hi, @steffenlarsen and @intel/llvm-reviewers-runtime Thanks very much. |
Hi, @steffenlarsen and @intel/llvm-reviewers-runtime |
Signed-off-by: jinge90 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Failures are unrelated. Linux Level-Zero failure: Windows OpenCL failure: |
Signed-off-by: jinge90 <[email protected]>
…est-suite#1301) Signed-off-by: jinge90 <[email protected]>
Signed-off-by: jinge90 [email protected]