Perform Valgrind checks with minimal inlining

Disable LTO and use `-Og` for Valgrind and Helgrind builds, so that we
can avoid making our Valgrind suppressions any more broad than
necessary.

Signed-off-by: Matt Wozniski <mwozniski@bloomberg.net>

.github/workflows/build.yml

@@ -123,6 +123,8 @@ jobs:
         python3 -m pip install --upgrade pip
         python3 -m pip install -r requirements-test.txt
         python3 -m pip install -e .
+      env:
+        MEMRAY_MINIMIZE_INLINING: 1
     - name: Run Valgrind
       run: make valgrind
     - name: Run Helgrind

setup.py

@@ -121,6 +121,8 @@ def build_js_files(self):
 if os.getenv("CYTHON_TEST_MACROS", None) is not None:
     TEST_BUILD = True
 
+MINIMIZE_INLINING = os.getenv("MEMRAY_MINIMIZE_INLINING", "") != ""
+
 COMPILER_DIRECTIVES = {
     "language_level": 3,
     "embedsignature": True,
@@ -133,8 +135,15 @@ def build_js_files(self):
     "c_string_encoding": "utf8",
 }
 EXTRA_COMPILE_ARGS = []
+EXTRA_LINK_ARGS = []
 UNDEF_MACROS = []
 
+if MINIMIZE_INLINING:
+    EXTRA_COMPILE_ARGS.append("-Og")
+else:
+    EXTRA_COMPILE_ARGS.append("-flto")
+    EXTRA_LINK_ARGS.append("-flto")
+
 # For Python 3.9+, hide all of our symbols except the module init function. For
 # Python 3.8 and earlier this isn't as easy, because PyMODINIT_FUNC doesn't
 # include __attribute__((visibility ("default"))), and Cython doesn't give us
@@ -204,8 +213,8 @@ def build_js_files(self):
     library_dirs=[str(LIBBACKTRACE_LIBDIR)],
     include_dirs=["src", str(LIBBACKTRACE_INCLUDEDIRS)],
     language="c++",
-    extra_compile_args=["-std=c++17", "-Wall", "-flto", *EXTRA_COMPILE_ARGS],
-    extra_link_args=["-std=c++17", "-flto", "-l:libbacktrace.a"],
+    extra_compile_args=["-std=c++17", "-Wall", *EXTRA_COMPILE_ARGS],
+    extra_link_args=["-std=c++17", "-l:libbacktrace.a", *EXTRA_LINK_ARGS],
     define_macros=DEFINE_MACROS,
     undef_macros=UNDEF_MACROS,
 )

valgrind.supp

@@ -81,7 +81,7 @@
    <cython_cached_constants>
    Memcheck:Leak
    ...
-   fun:__Pyx_InitCachedConstants
+   fun:*__Pyx_InitCachedConstants*
    ...
 }