Summary: | dev-qt/qtcore: Build uses -msse2 -mfpmath=sse on machines without sse2 | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Calvin Walton <calvin.walton> |
Component: | Current packages | Assignee: | Qt Bug Alias <qt> |
Status: | UNCONFIRMED --- | ||
Severity: | normal | CC: | ionen, sam, vanilkovy.puding |
Priority: | Normal | Keywords: | PATCH |
Version: | unspecified | ||
Hardware: | x86 | ||
OS: | Linux | ||
See Also: |
https://bugs.gentoo.org/show_bug.cgi?id=552942 https://bugs.gentoo.org/show_bug.cgi?id=775650 https://bugs.gentoo.org/show_bug.cgi?id=648004 |
||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | proposed patch of qtcore ebuild |
Description
Calvin Walton
2021-02-27 00:26:39 UTC
Created attachment 881374 [details, diff]
proposed patch of qtcore ebuild
I am also affected by this bug when trying to compile qtgui for AMD K6-2. Attached modification of qtcore-5.15.11-r1.ebuild allowed me to compile qtcore and then qtgui without failure.
As I thought about it a little more I think better solution is to modify eclass/qt5-build.eclass file. I added just one check into qt5_base_configure() function an got no problem to build and run QT libraries ever since. ... # MIPS DSP instruction set extensions $(is-flagq -mno-dsp && echo -no-mips_dsp) $(is-flagq -mno-dspr2 && echo -no-mips_dspr2) + $(tc-cpp-is-true "defined(__SSE2__)" ${CFLAGS} ${CXXFLAGS} || echo -no-feature-sse2) # use pkg-config to detect include and library paths -pkg-config ... Looks good to me. (In reply to vanilkovy.puding from comment #2) > + $(tc-cpp-is-true "defined(__SSE2__)" ${CFLAGS} ${CXXFLAGS} > || echo -no-feature-sse2) fwiw already doing something similar (testing with cpp) in qtbase and qt6-build for other reasons, so I wouldn't see a problem with this USE works too if preferred though, in Qt6 USE would've been more involved/messy and then still need to match with what users have in their cxxflags either way. (In reply to Sam James from comment #3) > Looks good to me. So can this be merged then? |