[PATCH] D39793: [asan] Remove semicolon after do {} while (0)

2017-11-08 Thread Tom de Vries via Phabricator via cfe-commits 
vries created this revision.
Herald added a subscriber: kubamracek.

this removes a semicolon after "do {} while (0)" in CHECK_SMALL_REGION. This 
allows the macro to be used in if-then-elses without curly braces.

[ https://gcc.gnu.org/ml/gcc-patches/2017-11/msg00298.html ]


https://reviews.llvm.org/D39793

Files:
  lib/asan/asan_poisoning.cc


Index: lib/asan/asan_poisoning.cc
===
--- lib/asan/asan_poisoning.cc
+++ lib/asan/asan_poisoning.cc
@@ -217,7 +217,7 @@
   uptr __bad = __asan_region_is_poisoned(__p, __size);\
   __asan_report_error(pc, bp, sp, __bad, isWrite, __size, 0);\
 } \
-  } while (false);\
+  } while (false)
 
 
 extern "C" SANITIZER_INTERFACE_ATTRIBUTE


Index: lib/asan/asan_poisoning.cc
===
--- lib/asan/asan_poisoning.cc
+++ lib/asan/asan_poisoning.cc
@@ -217,7 +217,7 @@
   uptr __bad = __asan_region_is_poisoned(__p, __size);\
   __asan_report_error(pc, bp, sp, __bad, isWrite, __size, 0);\
 } \
-  } while (false);\
+  } while (false)
 
 
 extern "C" SANITIZER_INTERFACE_ATTRIBUTE
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D39793: [asan] Remove semicolon after do {} while (0)

2017-11-13 Thread Tom de Vries via Phabricator via cfe-commits 
vries added a comment.

Committed.


https://reviews.llvm.org/D39793



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits