Am 2004.10.16 00:59:07 +0200 schrieb(en) Ian Romanick:Andreas Stenglein wrote:
Could someone else with an R100 test? see attached patch ( some changes to make mesa compile with gcc2.95.3 omitted)
What were those changes? Were they to avoid the SSE2 code?
1) basically some brackets{} , see attached (complete) patch. 2) no, but sse2 code won't run on my box. Only x86/MMX+/3DNow!+/SSE
Okay, those changes make sense. I guess the version of GCC that I'm using allows variables to be declared anywhere (like C++). I was just suspicious that some versions of binutils might support SSE but not SSE2. I can see that's not the case here.
In any case, your patch looks good to me. In fact, it looks almost identical to one that I'm about to commit to the MGA driver. ;)
since the old r200 and radeon code was identical, it is identical to your r200 patch, I only changed "r200" -> "radeon" Unfortunately it doesnt work well on my box when applied to mesa-cvs HEAD.
I fixed a couple bugs on Friday that would cause the code to not work in 16-bit mode. Basically, it would try to ues the 8888 SSE / MMX routines even in 16-bit mode. Oops. :)
------------------------------------------------------- This SF.net email is sponsored by: IT Product Guide on ITManagersJournal Use IT products in your business? Tell us what you think of them. Give us Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more http://productguide.itmanagersjournal.com/guidepromo.tmpl -- _______________________________________________ Dri-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/dri-devel
