]> git.ipfire.org Git - thirdparty/openembedded/openembedded-core.git/commit
perf: lift TARGET_CC_ARCH modification out of security_flags.inc
authorRasmus Villemoes <rasmus.villemoes@prevas.dk>
Wed, 13 Dec 2023 03:34:09 +0000 (11:34 +0800)
committerSteve Sakoman <steve@sakoman.com>
Tue, 19 Dec 2023 13:42:26 +0000 (03:42 -1000)
commit34f2f67055a9a29765629abef28ffa6b66c8ed41
tree8c9d64fb782f01af9f655fb75a1637bf658fdeb3
parent8408a44331a0c9e5ea86d822cebec06dcf495694
perf: lift TARGET_CC_ARCH modification out of security_flags.inc

Building perf without security_flags.inc being included in one's
distro results in the buildpaths warning

WARNING: perf-1.0-r9 do_package_qa: QA Issue: File /usr/bin/trace in
package perf contains reference to TMPDIR

because the ${DEBUG_PREFIX_MAP} does not get used. Most recipes get
that from CFLAGS, but the perf recipe explicitly unsets that.

Now ${SELECTED_OPTIMIZATION} of course contains more than just
${DEBUG_FLAGS}/${DEBUG_PREFIX_MAP}. For most TUs, perf's build system
adds its own optimization flags (-O6 for odd reasons), so for those
including the -O2 or -Og doesn't change anything. But looking at the
.o.cmd files show that there are some TUs which currently get built
without any -O flag. So for those adding the distro's
SELECTED_OPTIMIZATION seem to be the right thing to do.

(cherry-picked from Oe-Core rev: aa01c9122ef4a2159df503ef6ed25e802277f13a)

Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Lee Chee Yang <chee.yang.lee@intel.com>
Signed-off-by: Steve Sakoman <steve@sakoman.com>
meta/conf/distro/include/security_flags.inc
meta/recipes-kernel/perf/perf.bb