[Mesa-dev] [PATCH v2 2/2] etnaviv: annotate variables only used in debug build

Christian Gmeiner christian.gmeiner at gmail.com
Fri Dec 28 09:43:53 UTC 2018


Am Mi., 19. Dez. 2018 um 16:27 Uhr schrieb Lucas Stach <l.stach at pengutronix.de>:
>
> Some of the status variables in the compiler are only used in asserts
> and thus may be unused in release builds. Annotate them accordingly
> to avoid 'unused but set' warnings from the compiler.
>
> Signed-off-by: Lucas Stach <l.stach at pengutronix.de>

Reviewed-by: Christian Gmeiner <christian.gmeiner at gmail.com>

> ---
> v2: get rid of superfluous variable initialization
> ---
>  src/gallium/drivers/etnaviv/etnaviv_compiler.c | 11 ++++-------
>  1 file changed, 4 insertions(+), 7 deletions(-)
>
> diff --git a/src/gallium/drivers/etnaviv/etnaviv_compiler.c b/src/gallium/drivers/etnaviv/etnaviv_compiler.c
> index bbc61a59fc67..ceca5b8af997 100644
> --- a/src/gallium/drivers/etnaviv/etnaviv_compiler.c
> +++ b/src/gallium/drivers/etnaviv/etnaviv_compiler.c
> @@ -477,8 +477,7 @@ static void
>  etna_compile_parse_declarations(struct etna_compile *c)
>  {
>     struct tgsi_parse_context ctx = { };
> -   unsigned status = TGSI_PARSE_OK;
> -   status = tgsi_parse_init(&ctx, c->tokens);
> +   MAYBE_UNUSED unsigned status = tgsi_parse_init(&ctx, c->tokens);
>     assert(status == TGSI_PARSE_OK);
>
>     while (!tgsi_parse_end_of_tokens(&ctx)) {
> @@ -530,8 +529,7 @@ static void
>  etna_compile_pass_check_usage(struct etna_compile *c)
>  {
>     struct tgsi_parse_context ctx = { };
> -   unsigned status = TGSI_PARSE_OK;
> -   status = tgsi_parse_init(&ctx, c->tokens);
> +   MAYBE_UNUSED unsigned status = tgsi_parse_init(&ctx, c->tokens);
>     assert(status == TGSI_PARSE_OK);
>
>     for (int idx = 0; idx < c->total_decls; ++idx) {
> @@ -662,8 +660,7 @@ etna_compile_pass_optimize_outputs(struct etna_compile *c)
>  {
>     struct tgsi_parse_context ctx = { };
>     int inst_idx = 0;
> -   unsigned status = TGSI_PARSE_OK;
> -   status = tgsi_parse_init(&ctx, c->tokens);
> +   MAYBE_UNUSED unsigned status = tgsi_parse_init(&ctx, c->tokens);
>     assert(status == TGSI_PARSE_OK);
>
>     while (!tgsi_parse_end_of_tokens(&ctx)) {
> @@ -1812,7 +1809,7 @@ static void
>  etna_compile_pass_generate_code(struct etna_compile *c)
>  {
>     struct tgsi_parse_context ctx = { };
> -   unsigned status = tgsi_parse_init(&ctx, c->tokens);
> +   MAYBE_UNUSED unsigned status = tgsi_parse_init(&ctx, c->tokens);
>     assert(status == TGSI_PARSE_OK);
>
>     int inst_idx = 0;
> --
> 2.19.1
>
> _______________________________________________
> etnaviv mailing list
> etnaviv at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/etnaviv



-- 
greets
--
Christian Gmeiner, MSc

https://christian-gmeiner.info


More information about the mesa-dev mailing list