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
tweak PETSc tests #15767
tweak PETSc tests #15767
Conversation
// This test requires PETSc to be configured with the option | ||
// "--with-threadsafety" in case of debug builds of PETSc. | ||
// For optimized builds, the above option is needed only in case | ||
// users want PETSc to produce useful logs with "-log_view" runtime | ||
// option | ||
#if !defined(PETSC_USE_DEBUG) || defined(PETSC_HAVE_THREADSAFETY) |
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.
This will make the test fail if that flag is not set (because the output is different), right? If so, I would prefer to just abort the test.
What happens if PETSC_USE_DEBUG
is defined?
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.
The test won't fail since it will print zero as error
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.
With PETSC_USE_DEBUG
petsc uses additional non threadsafe structures to track entry and exit points of functions
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.
Oh, I see now -- the correct output is 0. Yes, this will work then.
Is PETSC_USE_DEBUG
the default?
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.
sadly yes
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.
Ah, bummer :-(
/rebuild |
No description provided.