Conversation
| } | ||
|
|
||
| #if idppc || idppc64 | ||
| static void Sys_GetProcessorId( char *vendor ) |
There was a problem hiding this comment.
To address the build error:
/home/tle/Work/ETe/src/qcommon/common.c:3622:13: error: redefinition of ‘Sys_GetProcessorId’
3622 | static void Sys_GetProcessorId( char *vendor )
| ^~~~~~~~~~~~~~~~~~
| elseif(X86_64) | ||
| set(compiler_flags_release ${compiler_flags_global}) | ||
| elseif(PPC64LE) | ||
| set(compiler_flags_release ${compiler_flags_global} -mcpu=power8) |
There was a problem hiding this comment.
POWER8 is the first to support Little Endian. This is the lowest common value for all CPUs that supports LE for example POWER8, POWER9 and POWER10
ref: https://www.ibm.com/docs/en/openxl-c-and-cpp-aix/17.1.0?topic=options-mcpu
| elseif(PPC64LE) | ||
| set(compiler_flags_release ${compiler_flags_global} -mcpu=power8) | ||
| elseif(PPC64) | ||
| set(compiler_flags_release ${compiler_flags_global} -mcpu=powerpc64) |
There was a problem hiding this comment.
Fallback to generic 64bit PPC64 if not LE
|
I will consider it but keep in mind that there will be zero support on any 3rd party servers or existing mods that lack ppc/ppc64 support as ET does not use QVM like Q3A. Also is there a reason you need to "force" with -mcpu? Shouldn't it be active if compiling on such target machine? |
Yes I am fully aware of that. This is mainly for PowerPC community to host their own LAN games
Well, if not specified, the value of |
Changes