lineage-rockchip/android_external_minigbm
1
0
Fork 0
Code Activity Actions

ANDROID: Move -fvisibility-inlines-hidden to CXXFLAGS only

Browse source 
Seeing this error in Android when running `cargo_embargo`:

error: command-line option ‘-fvisibility-inlines-hidden’ is valid
       for C++/ObjC++ but not for C [-Werror]

Test: cargo_embargo generate
Bug: none
Change-Id: Iddfaac9a2e4cce22401b65515dea3ed581e57400
This commit is contained in:
Devin Moore 2024-08-05 22:43:46 +00:00
parent 197d810946
commit ca50f86101
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
common.mk
View file

@ -323,9 +323,10 @@ COMMON_CFLAGS-clang := -fvisibility=hidden -ggdb \
COMMON_CFLAGS := -Wall -Wunused -Wno-unused-parameter \
-Wbool-operation -Wstring-compare -Wxor-used-as-pow \
-Wint-in-bool-context -Wfree-nonheap-object \
-Werror -Wformat=2 -fno-strict-aliasing -fvisibility-inlines-hidden \
-Werror -Wformat=2 -fno-strict-aliasing \
$(SSP_CFLAGS) -O1
CXXFLAGS += $(COMMON_CFLAGS) $(COMMON_CFLAGS-$(CXXDRIVER)) -std=gnu++20
CXXFLAGS += $(COMMON_CFLAGS) $(COMMON_CFLAGS-$(CXXDRIVER)) -std=gnu++20 \
-fvisibility-inlines-hidden
CFLAGS += $(COMMON_CFLAGS) $(COMMON_CFLAGS-$(CDRIVER)) -std=gnu17
# We undefine _FORTIFY_SOURCE because some distros enable it by default in
# their toolchains. This makes the compiler issue warnings about redefines