-
-
Notifications
You must be signed in to change notification settings - Fork 265
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add all float types to test #3501
Conversation
if (H5Dclose(dataset) < 0) | ||
goto error; | ||
} | ||
#if 0 |
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.
Why the #if 0
?
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.
And does this pass on Power? We have frequent long double issues there.
goto error; | ||
} | ||
#if 0 | ||
#if H5_SIZEOF_LONG_DOUBLE != H5_SIZEOF_DOUBLE |
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.
I feel like we should run this even when long double and double are the same size.
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.
#if 0 because this is the test that needs fixed - the orig_data values need to be appropriate for long double (they are just copies of double from above).
Close in favor of #4060 |
the 128 bit float test needs fixed