Skip to content

Commit 68cebed

Browse files
committed
assimp: work around gcc bug on SuperH
gcc versions earlier than gcc 6.x fail to build assimp on SuperH when static linking: AssxmlExporter.cpp:623:1: error: unable to find a register to spill in class 'GENERAL_REGS' It's the combination of -Os and *not* having -fPIC that makes gcc fail, which explains why configurations with dynamic linking work fine. -Os -fPIC -> works -Os -> fails -O2 -fPIC -> works -O2 -> works Therefore, as a workaround, we are forcing the use of -O2 on SuperH when the gcc version is older than gcc 6.x and we're statically linking. Fixes: http://autobuild.buildroot.net/results/ec88aa8118179e30e24603cc45292047dca19216/ Signed-off-by: Thomas Petazzoni <[email protected]>
1 parent 0fae861 commit 68cebed

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

package/assimp/assimp.mk

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,14 @@ ifeq ($(BR2_m68k),y)
1616
ASSIMP_CXXFLAGS += -mxgot
1717
endif
1818

19+
# workaround SuperH compiler failure when static linking (i.e -fPIC is
20+
# not passed) in gcc versions 5.x or older. The -Os optimization level
21+
# causes a "unable to find a register to spill in class
22+
# ‘GENERAL_REGS’" error. -O2 works fine.
23+
ifeq ($(BR2_sh):$(BR2_STATIC_LIBS):$(BR2_HOST_GCC_AT_LEAST_6),y:y:)
24+
ASSIMP_CXXFLAGS += -O2
25+
endif
26+
1927
ASSIMP_CONF_OPTS += -DASSIMP_BUILD_TESTS=OFF \
2028
-DCMAKE_CXX_FLAGS="$(TARGET_CXXFLAGS) $(ASSIMP_CXXFLAGS)"
2129

0 commit comments

Comments
 (0)