[Bug 2455] Review request: pcsx2 - A Sony Playstation2 emulator

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2455

--- Comment #24 from Andrea Musuruane <musuruan at gmail.com> 2012-09-14 15:45:28 CEST ---
(In reply to comment #21)
> >> The easiest way is to do as Richard said (just one parameter in cmake)
> For me it is not valid, it doesn't do what you think.
[..]

You are right. There is no RelWithDebInfo option. But it could be a nice future
addition ;-)

> For the parameter you can drop -m32 (only useful on 64 bits machine anyway) no
> impact. I'm not sure i686 is important but is there any sse2 cpu that doesn't
> support i686 instruction set? PCSX2 requires a powerful PC it won't run on 10
> years old PC anyway. Checked i686 is 1995 so 17 years old! It is older than the
> PS2 ;)

These are the flags actually used by Fedora to build for i686:
CFLAGS='-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
-fstack-protector --param=ssp-buffer-size=4  -m32 -march=i686 -mtune=atom
-fasynchronous-unwind-tables'

They are mandatory. This means you have to build Fedora packages with these
flags. As you can see they already include "-m32 -march=i686". "--march=i686"
implies only Pentium Pro and does not imply MMX or anything newer:
https://fedoraproject.org/wiki/Features/F12X86Support

Not all i686 processors, like the AMD Geode LX (and this is currently shipped),
support SSE or SSE2. And this is a problem for us.

> As upstream I will try to post somewhere a tar gz.

Thanks!

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.