mem: add cache class destructor to avoid memory leak
Review Request #1796 - Created March 27, 2013 and submitted
| Information | |
|---|---|
| Xiangyu Dong | |
| gem5 | |
| default | |
| Reviewers | |
| Default | |
Changeset 9618:0d84548ef3e3 --------------------------- mem: add cache class destructor to avoid memory leak Make valgrind a little bit happier (a test patch from a first-time code contributor)
Posted (March 27, 2013, 8:08 p.m.)
Welcome to the reviewboard :-) Great to see more contributors. I'm not sure how much this helps. With so many classes still not having proper destructors, is one class really making a difference in the output?
While it doesn't help much, it is a step in the right direction, so I think we should commit it. Simply for debugging memory leaks it would be nice to have all the classes have proper destructors.
Could you mark this as submitted? Thanks again :-)