My preference would be not to use static variables in pipe_screen
functions, because all functions needs to be thread-safe, but anyway:
Reviewed-by: Marek Olšák <marek.ol...@amd.com>
Marek
On Thu, Jan 26, 2017 at 7:27 PM, Emil Velikov <emil.l.veli...@gmail.com> wrote:
> From: Emil Velikov <emil.veli...@collabora.com>
>
> Provides quick and direct feedback to the user/developer.
>
> Cc: Marek Olšák <marek.ol...@amd.com>
> Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
> ---
> src/gallium/drivers/r300/r300_screen.c | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/r300/r300_screen.c
> b/src/gallium/drivers/r300/r300_screen.c
> index e5e7535358..12b94723cb 100644
> --- a/src/gallium/drivers/r300/r300_screen.c
> +++ b/src/gallium/drivers/r300/r300_screen.c
> @@ -82,8 +82,16 @@ static const char* chip_families[] = {
> static const char* r300_get_name(struct pipe_screen* pscreen)
> {
> struct r300_screen* r300screen = r300_screen(pscreen);
> + static char buffer[128];
> + const char *llvm = "";
>
> - return chip_families[r300screen->caps.family];
> +#ifdef HAVE_LLVM
> + llvm = " LLVM";
> +#endif
> +
> + util_snprintf(buffer, sizeof(buffer), "%s%s",
> + chip_families[r300screen->caps.family], llvm);
> + return buffer;
> }
>
> static int r300_get_param(struct pipe_screen* pscreen, enum pipe_cap param)
> --
> 2.11.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
