Fixing a double free issue with GCC > 9 in optimized mode

[mahmoodn]

This pull request is a fix for issues #32 and #49. With GCC > 9 the simulator crashes in trace_driven.cc. The bug can be bypassed with temporarily removing -O3 in trace-driven/Makefile. This function is supposed to return a pointer but it doesn't return anything indeed. Changing that to void fixes the runtime crash.

Additionally, by looking at get_next_threadblock_traces() there are unnecessary statements and return types:

• The clear() is redundant because that is done in trace_parser::get_next_threadblock_traces().
• trace_parser::get_next_threadblock_traces() always return true. So what is the need to return a boolean value?
• trace_kernel_info_t::get_next_threadblock_traces() also returns a boolean value which is not clear why because trace_kernel.get_next_threadblock_traces(threadblock_traces); doesn't use that.

[JRPan]

Hi,

Thank you for fixing that. Could you please merge this into dev branch instead of release? Thank you.

[mahmoodn]

It seems that current dev version is different from what I modified. I have to check if the problem exists with the new dev version.

[JRPan]

This has been merged into dev with #83. Closing this