[Mesa-dev] [PATCH] nouveau: Fix clang reserved-user-defined-literal error.
Samuel Pitoiset
samuel.pitoiset at gmail.com
Wed Mar 9 10:16:45 UTC 2016
Nouveau doesn't use c++11 except the codegen part.
How do you hit that issue? Pretty sure that you forced c++11, right?
I can't reproduce that compilation error with clang 3.9 btw.
On 03/09/2016 09:57 AM, Vinson Lee wrote:
> CXX codegen/nv50_ir.lo
> In file included from codegen/nv50_ir.cpp:28:
> ./nouveau_debug.h:19:30: error: invalid suffix on literal; C++11 requires a space between literal and identifier
> [-Wreserved-user-defined-literal]
> fprintf(stderr, "%s:%d - "fmt, __FUNCTION__, __LINE__, ##args)
> ^
>
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
> src/gallium/drivers/nouveau/nouveau_debug.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/nouveau/nouveau_debug.h b/src/gallium/drivers/nouveau/nouveau_debug.h
> index d17df81..546a4ad 100644
> --- a/src/gallium/drivers/nouveau/nouveau_debug.h
> +++ b/src/gallium/drivers/nouveau/nouveau_debug.h
> @@ -16,7 +16,7 @@
> #define NOUVEAU_DEBUG 0
>
> #define NOUVEAU_ERR(fmt, args...) \
> - fprintf(stderr, "%s:%d - "fmt, __FUNCTION__, __LINE__, ##args)
> + fprintf(stderr, "%s:%d - " fmt, __FUNCTION__, __LINE__, ##args)
>
> #define NOUVEAU_DBG(ch, args...) \
> if ((NOUVEAU_DEBUG) & (NOUVEAU_DEBUG_##ch)) \
>
--
-Samuel
More information about the mesa-dev
mailing list