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

[SYCL] Disable Signbit issue for NAN #653

Merged
merged 4 commits into from
Jan 12, 2022

Conversation

jinge90
Copy link

@jinge90 jinge90 commented Dec 22, 2021

Disabled test cases impacted by the fact that Clang and MSVC compilers have different behavior for sign bit of NAN/-NAN.

Signed-off-by: jinge90 [email protected]

@jinge90 jinge90 requested a review from vzakhari as a code owner December 22, 2021 02:26
@jinge90 jinge90 requested a review from vladimirlaz December 22, 2021 02:52
@jinge90
Copy link
Author

jinge90 commented Dec 22, 2021

Hi, @vladimirlaz
It seems that clang-format tool crashed when running format check for this patch, is it a know issue?

Thanks very much.

Copy link

@vladimirlaz vladimirlaz left a comment

Choose a reason for hiding this comment

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

Could you apply clang-format locally?

@jinge90
Copy link
Author

jinge90 commented Dec 28, 2021

Could you apply clang-format locally?

Hi, @vladimirlaz
I have applied clang-format locally but the patch still lead to clang-format crash in pre-ci environment.

I just used "clang-format-9 cmath_fp64_test.cpp" to get the clang formatted patch.

Thanks very much.

@jinge90 jinge90 requested a review from a team as a code owner January 11, 2022 13:11
@jinge90
Copy link
Author

jinge90 commented Jan 12, 2022

Hi, @intel/llvm-reviewers-runtime @vladimirlaz @vzakhari
Could you help review this patch? We need it to be merged as soon as possible.
Thanks very much.

@vladimirlaz vladimirlaz changed the title Disable Signbit issue for NAN [SYCL] Disable Signbit issue for NAN Jan 12, 2022
@vladimirlaz
Copy link

@jinge90 it looks like test coverage has been reduced. Could you please provide motivation in the PR description?

@vladimirlaz vladimirlaz merged commit 4b57f77 into intel:intel Jan 12, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Apr 12, 2022
Disabled test cases impacted by the fact that Clang and MSVC compilers have different behavior for sign bit of NAN/-NAN.

Signed-off-by: jinge90 <[email protected]>
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Disabled test cases impacted by the fact that Clang and MSVC compilers have different behavior for sign bit of NAN/-NAN.

Signed-off-by: jinge90 <[email protected]>
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.

2 participants