[Mesa-dev] [PATCH 09/12] i965: Make it possible to create a cfg_t without a backend_visitor.

Eric Anholt eric at anholt.net
Mon Nov 26 10:54:09 PST 2012


Kenneth Graunke <kenneth at whitecape.org> writes:

> All we really need is a memory context and the instruction list; passing
> a backend_visitor is just convenient at times.
>
> This will be necessary two patches from now.
> ---
>  src/mesa/drivers/dri/i965/brw_cfg.cpp | 17 ++++++++++++++---
>  src/mesa/drivers/dri/i965/brw_cfg.h   |  4 ++++
>  2 files changed, 18 insertions(+), 3 deletions(-)
>
> Technically I don't think we actually need to pass in a parent memory
> context since we allocate and free at ctor/dtor time anyway.  We might
> want to remove that and just pass the exec_list.

That would be fine with me.  We'd just need to be careful to free
cfg_ts.  There would be leak of one under INTEL_DEBUG=wm, though that's
the only one I see currently.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20121126/1d8b7be5/attachment.pgp>


More information about the mesa-dev mailing list