[Mesa-dev] [PATCH 1/9] mesa: Handle common extension checks with more compact code
Nanley Chery
nanleychery at gmail.com
Fri May 19 17:28:16 UTC 2017
On Fri, May 19, 2017 at 06:38:03AM -0700, Ian Romanick wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
>
> The previous code handled everything with the general case. I noticed
> that every time I converted an open-coded check to use a
> _mesa_has_EXT_foo() function, the text size of the driver increased.
>
> Almost all extensions only care what the current context API is, and
> the version does not matter. Handle those using more compact checks.
>
> text data bss dec hex filename
> 7037675 235248 37280 7310203 6f8b7b 32-bit i965_dri.so before
> 7034307 235248 37280 7306835 6f7e53 32-bit i965_dri.so after
> 6679695 303400 50608 7033703 6b5367 64-bit i965_dri.so before
> 6676143 303400 50608 7030151 6b4587 64-bit i965_dri.so after
>
Hi Ian,
I wrote a patch some time ago that reduces the cost of the extension
checks by a lot more with less code. The only thing I think may need
addressing is endianness. Would you consider using it instead if I
reworked it and sent it out to the list? You can find it here:
https://cgit.freedesktop.org/~nchery/mesa/commit/?h=1/ext/optimize&id=a02d88eba1d3129b27d3b5e6aaa976c3ca20cf79
Thanks,
Nanley
More information about the mesa-dev
mailing list