On 03/06/2015 05:34 AM, Emil Velikov wrote:
Replace the _MSC_VER >= 1200 with defined (_MSC_VER) and compact if/else
statements. We require MSVC 2008 or later with commit 46110c5d564.

Signed-off-by: Emil Velikov <[email protected]>
---
  src/util/macros.h | 8 +++-----
  1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/util/macros.h b/src/util/macros.h
index b862bfd..63daba3 100644
--- a/src/util/macros.h
+++ b/src/util/macros.h
@@ -73,15 +73,13 @@ do {                        \
     assert(!str);            \
     __builtin_unreachable(); \
  } while (0)
-#elif _MSC_VER >= 1200
+#elif defined (_MSC_VER)
  #define unreachable(str)    \
  do {                        \
     assert(!str);            \
     __assume(0);             \
  } while (0)
-#endif
-
-#ifndef unreachable
+#else
  #define unreachable(str) assert(!str)
  #endif

@@ -99,7 +97,7 @@ do {                       \
  #define assume(expr) ((expr) ? ((void) 0) \
                               : (assert(!"assumption failed"), \
                                  __builtin_unreachable()))
-#elif _MSC_VER >= 1200
+#elif defined (_MSC_VER)
  #define assume(expr) __assume(expr)
  #else
  #define assume(expr) assert(expr)


Building with this patch now and looks good so far.

Reviewed-by: Brian Paul <[email protected]>

_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to