Bug 2529 - Unbreak altivec build
Status: RESOLVED FIXED
Alias: None
Product: ioquake3
Classification: Unclassified
Component: Platform
Version: 1.33 SVN
Hardware: Macintosh MacOS X
: P2 normal
Assignee: Zachary J. Slater
QA Contact: ioquake3 bugzilla mailing list
URL:
: 2530
Depends on:
Blocks:
 
Reported: 2005-12-05 20:55 EST by Joshua Root
Modified: 2007-05-21 11:24:41 EDT
2 users (show)

See Also:



Description Joshua Root 2005-12-05 20:55:07 EST
LerpMeshVertexes_altivec still uses the ALIGN16 macro which as I understand it has been removed in favour of ALIGN(x) from q_shared.h. Here's a patch that makes things work again:

Index: code/renderer/tr_surface.c
===================================================================
--- code/renderer/tr_surface.c  (revision 419)
+++ code/renderer/tr_surface.c  (working copy)
@@ -615,10 +615,10 @@
 {
        short   *oldXyz, *newXyz, *oldNormals, *newNormals;
        float   *outXyz, *outNormal;
-       float   oldXyzScale ALIGN16;
-       float   newXyzScale ALIGN16;
-       float   oldNormalScale ALIGN16;
-       float newNormalScale ALIGN16;
+       float   oldXyzScale ALIGN(16);
+       float newXyzScale ALIGN(16);
+       float   oldNormalScale ALIGN(16);
+       float newNormalScale ALIGN(16);
        int             vertNum;
        unsigned lat, lng;
        int             numVerts;
Comment 1 Tim Angus 2005-12-06 13:18:00 EST
*** Bug 2530 has been marked as a duplicate of this bug. ***
Comment 2 Tim Angus 2005-12-06 13:19:42 EST
Fixed in 431. Cheers.
Comment 3 Ryan C. Gordon 2007-05-21 11:24:41 EDT
Setting a QA contact on all ioquake3 bugs, even resolved ones. Sorry if you get a flood of email from this, it should only happen once. Apologies for the incovenience.

--ryan.