"Kendall Bennett" <[EMAIL PROTECTED]> writes:

> Actually it appears to be more deep rooted than that. This macros 
> appears to only be used by the Intel SSE stuff, which is not yet 
> being used. If you look further down where it is used in the rest of 
> the code, it only passes two arguments to the function! I am not at 
> all familiar with this code, but I suspect the lines:
> 
> #define CMPEQPS(a, b)   cmpeqps P_ARG3(a, b)
> #define CMPLTPS(a, b)   cmpltps P_ARG3(a, b)
> 
> should be:
> 
> #define CMPEQPS(a, b)   cmpeqps P_ARG2(a, b)
> #define CMPLTPS(a, b)   cmpltps P_ARG2(a, b)
> 
> while:
> 
> #define SHUFPS(a, b, c) shufps P_ARG3(a, b, c)
> 
> will use the correct version of the P_ARG3 macro.
> 
> I don't want to change this as I have no way to test it. Perhaps 
> someone who is working on Intel SSE stuff can do that...

Sorry, I must have been asleep when I wrote that.  I'll fix it
tonight.


Josh



_______________________________________________
Mesa-dev maillist  -  [EMAIL PROTECTED]
http://lists.mesa3d.org/mailman/listinfo/mesa-dev

Reply via email to