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
Fix empty array handling in lu, rank, and qr. Other minor refactoring #2838
Conversation
umar456
commented
Apr 11, 2020
- some linear algebra functions could have leaked memory in case the input was null. This PR addresses that potential leak and handles the empty or null array case.
- Minor refactor in median
- Minor refactor in DefaultMemoryManager. Just changed names of the variables.
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.
Please also fix the formatting problem in the following files:
test/memory.cpp
test/median.cpp
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.
Memory leaks in all new tests for lu_dense.cpp
, qr_dense.cpp
and rank_dense.cpp
. I believe that is why the ASAN tests failed.
Also, the formatting is still off in test/memory.cpp
file.
Addressed all feedback |