[Mesa-dev] [PATCH V2] mesa: Permanently enable features supported by target CPU at compile time.

Siavash Eliasi siavashserver at gmail.com
Sat Nov 8 20:37:33 PST 2014


On 11/08/2014 09:55 PM, Emil Velikov wrote:
> A comment I could not withheld based on your earlier post - "We require
> micro-benchmark for this code. It will take me hours to find why mesa is
> so slow now :P"

Which brings the question why didn't you post to that thread/topic in 
first place instead :P

> Ideally mesa should have an infrastructure/farm that handles regressions
> - be that performance or otherwise. Pretty sure some companies have such
> features but those seem to be hidden behind locked doors :'(

Yes, that's unfortunate. But atleast we have Phoronix :)

> Can you confirm that it does not cause issues with "interesting" setups
> such as https://bugs.freedesktop.org/show_bug.cgi?id=71547

Challenge accepted! What my patch is doing is to check for provided 
compile flags (-msse, ...) on compile time (__SSE__, ...) and set 
"cpu_has_sse" macro to "1" which allows any sane compiler to turn this 
pieces of code:

#ifdef USE_SSE
if (cpu_has_sse)
{
/* SSE code path */
}
else
#endif
{
/* C fallback */
}

into this:

/* SSE code path */

by using compile time information by target CPU.

Best regards,
Siavash Eliasi.


More information about the mesa-dev mailing list