Closed iamluc closed 3 weeks ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 78.49%. Comparing base (
98839cf
) to head (72358cc
).
Benchmark execution time: 2024-09-10 16:36:55
Comparing candidate commit 72358ccbb69ee53d4a97d9d0a972cdf570d077a0 in PR branch luc/crashtracker-fix-use-after-free
with baseline commit 98839cf637cac6374236c4d40dc70bede9551472 in branch master
.
Found 6 performance improvements and 1 performance regressions! Performance is the same for 171 metrics, 0 unstable metrics.
execution_time
[-194.913µs; -62.487µs] or [-6.427%; -2.060%]execution_time
[-187.752µs; -79.208µs] or [-5.488%; -2.315%]execution_time
[+5.059µs; +7.361µs] or [+3.536%; +5.145%]execution_time
[-12.244µs; -7.576µs] or [-6.498%; -4.021%]execution_time
[-12.508µs; -9.592µs] or [-4.185%; -3.210%]execution_time
[-13.513µs; -9.134µs] or [-4.032%; -2.726%]execution_time
[-21.842ms; -21.618ms] or [-83.027%; -82.179%]Do you want to also toggle the default of the LOG_BACKTRACES setting if compiled with asan (via preprocessor directive)?
I find it easier to disable the entire signal handling. WDYT?
@iamluc Yeah, but then you can't run crashtracker tests with asan either and you'll have to disable tests via skipif too. I'd rather just add a define in configuration.h around the default value.
Description
Reviewer checklist