[Mesa-dev] [PATCH v2 0/3][RFC]mesa/st: glsl_to_tgsi: improved temp-reg lifetime estimation

Emil Velikov emil.l.velikov at gmail.com
Fri Jun 16 14:21:25 UTC 2017


Hi Gert,

Welcome to Mesa, and apologies for chiming in so late.

Please don't use STL within core mesa code. While some places do use
it, those are quite isolated and have specific role.
For example:
 - st/clover - heavily templated, pure C++
 - drivers/swr - as above
 - drivers/nouveau/codegen - mix of STL and local alternatives of STL
 - drivers/r600/sb - somewhat lightweight on STL usage.

A couple of additional ideas:
 - where possible try to split patches even further.
IIRC 2/3 adds some 2kloc, which may be hard to review properly.
 - do keep performance numbers within the commit summary.
This way the details are preserved in git log for future references.

Thanks
Emil


More information about the mesa-dev mailing list