[Mesa-dev] [PATCH] i965: Use rzalloc for cfg_t

[Marek Olšák]

Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek

On Wed, Nov 2, 2016 at 8:01 AM, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> Valgrind reports that we use cfg.cycle_count uninitialised, so zero the
> cfg_t on construction.
>
> Fixes: 52d2b28f7f10 ("ralloc: use rzalloc where it's necessary")
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Kenneth Graunke <kenneth at whitecape.org>
> Cc: Marek Olšák <mareko at kemper.freedesktop.org>
> Cc: Edmondo Tommasina <edmondo.tommasina at gmail.com>
> Cc: Nicolai Hähnle <nicolai.haehnle at amd.com>
> ---
>  src/mesa/drivers/dri/i965/brw_cfg.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_cfg.h b/src/mesa/drivers/dri/i965/brw_cfg.h
> index b8af40f..ec21ae7 100644
> --- a/src/mesa/drivers/dri/i965/brw_cfg.h
> +++ b/src/mesa/drivers/dri/i965/brw_cfg.h
> @@ -274,7 +274,7 @@ bblock_t::last_non_control_flow_inst()
>
>  struct cfg_t {
>  #ifdef __cplusplus
> -   DECLARE_RALLOC_CXX_OPERATORS(cfg_t)
> +   DECLARE_RZALLOC_CXX_OPERATORS(cfg_t)
>
>     cfg_t(exec_list *instructions);
>     ~cfg_t();
> --
> 2.10.2
>