[Mesa-dev] [PATCH] r600g: Only compare active vertex elements

Tilman Sauerbeck tilman at code-monkey.de
Mon Nov 22 11:36:31 PST 2010


Mathias Fröhlich [2010-11-22 19:30]:
 
> Attached is a patch to r600g which limits the number of compared vertex 
> elements to the amount that is actually active instead of comparing all 
> possible 32 vertex elements.
> The patch moves the r600_shader_update function down in my usual profile runs.
>
> Please review/apply.

Looks good. I wanted to do this change some time ago, but couldn't get
proof that it actually helped performance-wise.
Anyway, according to perf it helps with OA.

I'll apply the patch unless Dave or Jerome have objections.

Regards,
Tilman

-- 
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing on usenet and in e-mail?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20101122/760081c2/attachment-0001.pgp>


More information about the mesa-dev mailing list