rv32emu icon indicating copy to clipboard operation
rv32emu copied to clipboard

Introduce Valgrind to the CI pipeline

Open henrybear327 opened this issue 11 months ago • 6 comments

The static analyzer alone can't catch all the runtime issues. In this commit, the dynamic analysis tool Valgrind is added to the CI pipeline.

ENABLE_SDL is set to 0, in order to reduce noise caused by the external libraries.

Reference:

  • https://valgrind.org/docs/manual/quick-start.html
  • https://linux.die.net/man/1/gcc

henrybear327 avatar Mar 01 '24 20:03 henrybear327

To merge this pull request, all the memory-related leaks need to be addressed first.

Currently, based on my initial testing, when disabling SDL and JIT, no memory leaks are found. When enabling JIT, there will be plenty of leaks.

henrybear327 avatar Mar 01 '24 20:03 henrybear327