Bug#907411: pcsx2 FTBFS with gcc 8

[Juhani Numminen]

On Mon, 27 Aug 2018 20:55:56 +0300 Adrian Bunk <bunk at debian.org> wrote:
> Source: pcsx2
> Version: 1.4.0+dfsg-2
> Severity: serious
> Tags: ftbfs
> 
> https://tests.reproducible-builds.org/debian/rb-pkg/unstable/i386/pcsx2.html
> 
> ...
> In file included from /usr/lib/gcc/i686-linux-gnu/8/include/x86intrin.h:74,
>                  from /build/1st/pcsx2-1.4.0+dfsg/common/src/x86emitter/cpudetect.cpp:19:
> /usr/lib/gcc/i686-linux-gnu/8/include/xsaveintrin.h:60:1: error: ambiguating new declaration of 'long long int _xgetbv(unsigned int)'
>  _xgetbv (unsigned int __A)
>  ^~~~~~~
> In file included from /build/1st/pcsx2-1.4.0+dfsg/common/include/Pcsx2Defs.h:31,
>                  from /build/1st/pcsx2-1.4.0+dfsg/common/src/x86emitter/PrecompiledHeader.h:23,
>                  from /build/1st/pcsx2-1.4.0+dfsg/common/src/x86emitter/cpudetect.cpp:16:
> /build/1st/pcsx2-1.4.0+dfsg/common/include/intrin_x86.h:107:69: note: old declaration 'long long unsigned int _xgetbv(unsigned int)'
>  static __inline__ __attribute__((always_inline)) unsigned long long _xgetbv(unsigned int index)

This might be relevant:

"common: Use GCC's _xgetbv definition from GCC 8.2 onwards
The _xgetbv bug was fixed, so avoid using our own definition (again)."
https://github.com/PCSX2/pcsx2/commit/e8ed18febaa046a3383a4b960517372a5bc554d1


Regards,
Juhani