-
Notifications
You must be signed in to change notification settings - Fork 894
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
Remove checking for specific tests in memcheck script #16412
Remove checking for specific tests in memcheck script #16412
Conversation
@@ -50,6 +50,8 @@ CUDF_KERNEL void test_kernel(int* data) { data[threadIdx.x] = threadIdx.x; } | |||
// calls. | |||
TEST(StreamCheck, FailedKernel) | |||
{ | |||
if (getenv("LIBCUDF_MEMCHECK_ENABLED")) { return; } |
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.
perhaps we should use GTEST_SKIP()
rather than just returning?
if (getenv("LIBCUDF_MEMCHECK_ENABLED")) { return; } | |
if (getenv("LIBCUDF_MEMCHECK_ENABLED")) { GTEST_SKIP(); } |
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.
Ok. This works in this file but not in the test_default_stream_identification.cu
since that one is not actually a gtest.
/merge |
Description
Removes the checking for specific gtests in the
run_cudf_memcheck_ctests.sh
script. Each of those tests can check theLIBCUDF_MEMCHECK_ENABLED
environment variable themselves.This simplifies the script logic and may help with replacing this with ctest logic in the future.
Checklist