Closed sjsprecious closed 1 month ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 93.27%. Comparing base (
46d0a74
) to head (6206845
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@mattldawson @K20shores @mwaxmonsky I ran the GPU memory leak check for the CUDA integration test on Derecho's A100 GPU. It passed the check within 10963461 ms (~3 hours and 10 minutes).
This PR:
--leak-check full
forcompute-sanitizer
to check the memory leak on the GPU.cudaFree
for a bool pointer on the GPU.std::swap
rather thanstd::move
in the move constructor and assignment in a CUDA class.All the GPU unit tests are now passed for GPU memory check on Derecho's A100 GPU. The integration test is not checked for GPU memory leak because it takes too long.
fix #598