-
Notifications
You must be signed in to change notification settings - Fork 130
Conversation
Signed-off-by: jinge90 <[email protected]>
Hi, @vladimirlaz Thanks very much. |
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.
Could you apply clang-format locally?
Signed-off-by: jinge90 <[email protected]>
Hi, @vladimirlaz I just used "clang-format-9 cmath_fp64_test.cpp" to get the clang formatted patch. Thanks very much. |
Hi, @intel/llvm-reviewers-runtime @vladimirlaz @vzakhari |
@jinge90 it looks like test coverage has been reduced. Could you please provide motivation in the PR description? |
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]>
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]>
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]