From 7feac8bdb9d8f76f20e41e3b0314168738dd37fd Mon Sep 17 00:00:00 2001 From: Emil Velikov Date: Thu, 24 Nov 2016 18:18:13 +0000 Subject: [PATCH] anv: use do { } while (0) in the anv_finishme macro Use the generic construct instead of the currect GCC specific one. Suggested-by: Kenneth Graunke Signed-off-by: Emil Velikov Reviewed-by: Kenneth Graunke --- src/intel/vulkan/anv_private.h | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h index 2fc543daacc..edc008db715 100644 --- a/src/intel/vulkan/anv_private.h +++ b/src/intel/vulkan/anv_private.h @@ -207,13 +207,14 @@ void anv_loge_v(const char *format, va_list va); /** * Print a FINISHME message, including its source location. */ -#define anv_finishme(format, ...) ({ \ - static bool reported = false; \ - if (!reported) { \ - __anv_finishme(__FILE__, __LINE__, format, ##__VA_ARGS__); \ - reported = true; \ - } \ -}) +#define anv_finishme(format, ...) \ + do { \ + static bool reported = false; \ + if (!reported) { \ + __anv_finishme(__FILE__, __LINE__, format, ##__VA_ARGS__); \ + reported = true; \ + } \ + } while (0) /* A non-fatal assert. Useful for debugging. */ #ifdef DEBUG