Feature #375

Backport AltiVec optimizations from ioquake3

Added by BSzili almost 6 years ago. Updated over 5 years ago.

Status:New% Done:

60%

Priority:NormalSpent time:-
Assignee:BSzili
Category:Client
Target version:2.78
OS: Arch:

Description

ioquake3 has some AltiVec optimizations in the renderer, which could be useful now that MorphOS is supported.

Associated revisions

Revision 331bedf9
Added by BSzili almost 6 years ago

qcommon: added com_altivec cvar, refs #375

Revision c30f757a
Added by BSzili almost 6 years ago

renderer: altivec LerpMeshVertexes, refs #375

Revision 1d060fe4
Added by BSzili almost 6 years ago

renderer: altivec DynamicLightPass, refs #375

Revision 6fb8352e
Added by BSzili almost 6 years ago

client: don’t update com_altivec on non-ppc platforms, refs #375

Revision 2cc375c9
Added by IR4T4 over 5 years ago

client: don’t create com_altivec cvar if idppc macro isn’t set refs #375

History

#1 Updated by BSzili almost 6 years ago

  • Target version changed from 2.71 to 2.71rc4
  • % Done changed from 0 to 60

#2 Updated by IR4T4 over 5 years ago

Currently cvar com_altivec cvar is kind of messed ... macros 'idppc_altivec’, 'idppc’ and 'USE_RENDERER_DLOPEN’ are used to deal with that var. The var itself is always active - there is NO macro in common.c and common.h.

#3 Updated by IR4T4 over 5 years ago

I would like to find a solution to close this ticket. Radegast mentioned we shouldn’t add AltiVec optimizations/code anymore. BSzili, is there are real benefit for Pandora if we add this feature?

#4 Updated by BSzili over 5 years ago

IR4T4 wrote:

I would like to find a solution to close this ticket. Radegast mentioned we shouldn’t add AltiVec optimizations/code anymore.

I originally added these optimizations, because there was already some AltiVec optimized code in the mixer, so I assumed it’s OK to add them to the renderer too. Anyway I’m fine with removing them, because they only give a small boost in high resolutions.

BSzili, is there are real benefit for Pandora if we add this feature?

They are no use for Pandora, which has an ARM processor.

#5 Updated by IR4T4 over 5 years ago

  • Target version changed from 2.71rc4 to 2.78

Also available in: Atom PDF