Memory Leak in Timeline

cyfitech / cris-core

A library of inter-thread message passing, and more

Apache License 2.0

2 stars 0 forks source link

Memory Leak in Timeline #269

Open chenhao94 opened 1 month ago

chenhao94 commented 1 month ago

The global variables in src/utils/timeline.cc trigger memory leak detection on LLVM 15+, which blocks us from running ASAN on a newer toolchain.

See https://github.com/cyfitech/cris-core/actions/runs/9344436224/job/25715593728

xkszltl commented 1 month ago

Only add toolchain that works. Do not try to "swap" distro.
No need to change sanitizer version for now. It's not "newer is better".

chenhao94 commented 1 month ago

We have to upgrade soon if we are moving to Ubuntu 24.04. The default GCC version of it is 13, which only works with LLVM 15+. So I guess we don't have a choice of not upgrading it.

chenhao94 commented 1 month ago

Otherwise it will be hard to run asan tests on Ubuntu 24.04

xkszltl commented 1 month ago

What does "only works with" means?

chenhao94 commented 1 month ago

Clang 13 cannot compile GCC 13 headers, while GCC 13 is the default on this distro.