Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL][libdevice] Tests for half value type cast utils. #1301

Merged
merged 16 commits into from
Nov 3, 2022

Conversation

jinge90
Copy link

@jinge90 jinge90 commented Oct 2, 2022

Signed-off-by: jinge90 [email protected]

@jinge90 jinge90 requested a review from a team as a code owner October 2, 2022 09:23
@jinge90 jinge90 requested a review from steffenlarsen October 2, 2022 09:23
@jinge90 jinge90 marked this pull request as draft October 2, 2022 09:23
@jinge90
Copy link
Author

jinge90 commented Oct 2, 2022

This PR is test for intel/llvm#6930

}

{
std::initializer_list<uint64_t> input_vals = {

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)

Copy link
Author

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.

@jinge90 jinge90 marked this pull request as ready for review October 28, 2022 13:39
@jinge90
Copy link
Author

jinge90 commented Oct 28, 2022

Hi, @steffenlarsen and @intel/llvm-reviewers-runtime
There is one failure in pre-ci which is xpti/content.cpp, this failure is not related to this PR. Could you help review?

Thanks very much.

@jinge90
Copy link
Author

jinge90 commented Nov 1, 2022

Hi, @steffenlarsen and @intel/llvm-reviewers-runtime
Kind ping~

Copy link

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@steffenlarsen
Copy link

Failures are unrelated.

Linux Level-Zero failure:
SYCL :: Plugin/level_zero_interop_memcpy.cpp

Windows OpenCL failure:
SYCL :: KernelAndProgram/kernel-bundle-merge-options-env.cpp

@steffenlarsen steffenlarsen merged commit cb7d25a into intel:intel Nov 3, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants