diff options
| author | James Clark <[email protected]> | 2025-03-19 11:40:09 +0000 |
|---|---|---|
| committer | Namhyung Kim <[email protected]> | 2025-03-20 00:00:39 +0000 |
| commit | f5b07010c13c77541e8ade167d05bef3b8a63739 (patch) | |
| tree | 0d793ee57db4a5dcc36a5406d40d5d10b422e178 /tools/perf/scripts/python/export-to-sqlite.py | |
| parent | perf pmu: Handle memory failure in tool_pmu__new() (diff) | |
| download | kernel-f5b07010c13c77541e8ade167d05bef3b8a63739.tar.gz kernel-f5b07010c13c77541e8ade167d05bef3b8a63739.zip | |
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 <[email protected]>
Reviewed-by: Ian Rogers <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Namhyung Kim <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/export-to-sqlite.py')
0 files changed, 0 insertions, 0 deletions