[PATCH] Simplify and improve x86/amd64 CPU build flags #2329

New Issue

Closed

opened 2013-12-29 12:40:29 +00:00 by fabio · 6 comments

fabio commented

2013-12-29 12:40:29 +00:00

The attached patch does the following:

on amd64 removes the "-msse" flag, which is already enabled by default;
on x86 changes the flags from "-march=i686 -msse" to "-march=pentium3 -mtune=generic". This should also enable other CPU features like MMX, while keeping the same minimun supported CPU (Intel Pentium3 or AMD Athlon 4) previously required by "-msse".

Tested on x86 (Core2 Duo).

The attached patch does the following: * on amd64 removes the "-msse" flag, which is already enabled by default; * on x86 changes the flags from "-march=i686 -msse" to "-march=pentium3 -mtune=generic". This should also enable other CPU features like MMX, while keeping the same minimun supported CPU (Intel Pentium3 or AMD Athlon 4) previously required by "-msse". Tested on x86 (Core2 Duo).

fabio added the

Build & Packages

labels 2013-12-29 12:40:29 +00:00

fabio added this to the Alpha 16 milestone 2013-12-29 12:40:29 +00:00

fabio commented

2013-12-29 12:41:12 +00:00

Author

Attachment cpu-flags.diff (1178 bytes) added

actual patch

**Attachment** cpu-flags.diff (1178 bytes) added actual patch

cpu-flags.diff

1.2 KiB

fabio commented

2014-05-04 11:50:18 +00:00

Author

Better to postpone this after a new release.

Better to postpone this after a new release.

fabio modified the milestone from Alpha 16 to Alpha 17

2014-05-04 11:50:18 +00:00

JoshuaJB commented

2014-05-06 19:32:08 +00:00

It should be fine to commit this as soon as the Alpha 16 freeze is over.

It should be fine to commit this as soon as the Alpha 16 freeze is over.

fabio commented

2014-05-19 12:19:01 +00:00

Author

See 9f86df78d9

Simplify and improve x86/amd64 CPU build flag

On amd64 removes the "-msse" flag, which is already
enabled by default.

On x86 changes the flags from "-march=i686 -msse" to
"-march=pentium3 -mtune=generic". This should also enable
other CPU features like MMX, while keeping the same
minimun supported CPU (Intel Pentium3 or AMD Athlon 4)
previously required by "-msse". 

Fixes #2329.

See 9f86df78d9afae31b997feea4d4a9cf98ed3d363 ``` Simplify and improve x86/amd64 CPU build flag On amd64 removes the "-msse" flag, which is already enabled by default. On x86 changes the flags from "-march=i686 -msse" to "-march=pentium3 -mtune=generic". This should also enable other CPU features like MMX, while keeping the same minimun supported CPU (Intel Pentium3 or AMD Athlon 4) previously required by "-msse". Fixes #2329. ```

fabio added the

label 2014-05-19 12:19:01 +00:00

fabio self-assigned this 2014-05-19 12:19:01 +00:00

fabio closed this issue

2014-05-19 12:19:01 +00:00

raymond commented

2014-05-19 13:40:37 +00:00

Any performance improvements?
Why not: "‘core2’
Intel Core 2 CPU with 64-bit extensions, MMX, SSE, SSE2, SSE3 and SSSE3 instruction set support." ? Maybe "SSE2, SSE3 and SSSE" (in core2) improve the performance?
(https://gcc.gnu.org/onlinedocs/gcc-4.9.0/gcc/i386-and-x86-64-Options.html#i386-and-x86-64-Options)

Pentium III last produced until 2002.
Note: SSE II support: 99 %; SSE III support: 95% (http://feedback.wildfiregames.com/static/cpucaps.html)

So increasing the requirements to Core2Duo CPU is not a problem.

Any performance improvements? Why not: "‘core2’ Intel Core 2 CPU with 64-bit extensions, MMX, SSE, SSE2, SSE3 and SSSE3 instruction set support." ? Maybe "SSE2, SSE3 and SSSE" (in core2) improve the performance? (<https://gcc.gnu.org/onlinedocs/gcc-4.9.0/gcc/i386-and-x86-64-Options.html#i386-and-x86-64-Options>) Pentium III last produced until 2002. Note: SSE II support: 99 %; SSE III support: 95% (<http://feedback.wildfiregames.com/static/cpucaps.html>) So increasing the requirements to Core2Duo CPU is not a problem.

fabio commented

2014-05-19 14:35:57 +00:00

Author

See discussion on forum: http://www.wildfiregames.com/forum/index.php?showtopic=18672

See discussion on forum: <http://www.wildfiregames.com/forum/index.php?showtopic=18672>

Sign in to join this conversation.

No Label

1: Release Blocker

4: Nice To Have

5: If Time Permits

Art & Animation

Build & Packages

Internationalization & Localization

Multiplayer Lobby

Music & Sound FX

Non-game systems

UI – Game setup

UI – Miscellaneous

UI & Simulation

No Milestone

No project

No Assignees

3 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: wfg/0ad#2329

No description provided.