diff options
author | James Clark <james.clark@linaro.org> | 2025-03-19 14:40:09 +0300 |
---|---|---|
committer | Namhyung Kim <namhyung@kernel.org> | 2025-03-20 03:00:39 +0300 |
commit | f5b07010c13c77541e8ade167d05bef3b8a63739 (patch) | |
tree | 0d793ee57db4a5dcc36a5406d40d5d10b422e178 /tools/perf/scripts/python/task-analyzer.py | |
parent | 431db90a7303cb394c5a881b4479946f64052727 (diff) | |
download | linux-f5b07010c13c77541e8ade167d05bef3b8a63739.tar.xz |
libperf: Don't remove -g when EXTRA_CFLAGS are used
When using EXTRA_CFLAGS, for example "EXTRA_CFLAGS=-DREFCNT_CHECKING=1",
this construct stops setting -g which you'd expect would not be affected
by adding extra flags. Additionally, EXTRA_CFLAGS should be the last
thing to be appended so that it can be used to undo any defaults. And no
condition is required, just += appends to any existing CFLAGS and also
appends or doesn't append EXTRA_CFLAGS if they are or aren't set.
It's not clear why DEBUG=1 is required for -g in Perf when in libperf
it's always on, but I don't think we need to change that behavior now
because someone may be depending on it.
Signed-off-by: James Clark <james.clark@linaro.org>
Reviewed-by: Ian Rogers <irogers@google.com>
Link: https://lore.kernel.org/r/20250319114009.417865-1-james.clark@linaro.org
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python/task-analyzer.py')
0 files changed, 0 insertions, 0 deletions