Title: [206580] trunk/Source/WTF
- Revision
- 206580
- Author
- [email protected]
- Date
- 2016-09-29 08:12:54 -0700 (Thu, 29 Sep 2016)
Log Message
Clang 3.9 reports a compilation warning about ENABLE_EXCEPTION_SCOPE_VERIFICATION
https://bugs.webkit.org/show_bug.cgi?id=162718
Patch by Fujii Hironori <[email protected]> on 2016-09-29
Reviewed by Alex Christensen.
Clang 3.9 reports a following compilation warning:
Source/_javascript_Core/runtime/VM.h:656:5: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]
* wtf/Platform.h: Changed the definition of ENABLE_EXCEPTION_SCOPE_VERIFICATION not to use 'defined'.
Modified Paths
Diff
Modified: trunk/Source/WTF/ChangeLog (206579 => 206580)
--- trunk/Source/WTF/ChangeLog 2016-09-29 12:48:16 UTC (rev 206579)
+++ trunk/Source/WTF/ChangeLog 2016-09-29 15:12:54 UTC (rev 206580)
@@ -1,3 +1,15 @@
+2016-09-29 Fujii Hironori <[email protected]>
+
+ Clang 3.9 reports a compilation warning about ENABLE_EXCEPTION_SCOPE_VERIFICATION
+ https://bugs.webkit.org/show_bug.cgi?id=162718
+
+ Reviewed by Alex Christensen.
+
+ Clang 3.9 reports a following compilation warning:
+ Source/_javascript_Core/runtime/VM.h:656:5: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]
+
+ * wtf/Platform.h: Changed the definition of ENABLE_EXCEPTION_SCOPE_VERIFICATION not to use 'defined'.
+
2016-09-28 Mark Lam <[email protected]>
Re-enable StringView life-cycle checking.
Modified: trunk/Source/WTF/wtf/Platform.h (206579 => 206580)
--- trunk/Source/WTF/wtf/Platform.h 2016-09-29 12:48:16 UTC (rev 206579)
+++ trunk/Source/WTF/wtf/Platform.h 2016-09-29 15:12:54 UTC (rev 206580)
@@ -896,8 +896,12 @@
#endif
#ifndef ENABLE_EXCEPTION_SCOPE_VERIFICATION
-#define ENABLE_EXCEPTION_SCOPE_VERIFICATION (!defined(NDEBUG))
+#ifdef NDEBUG
+#define ENABLE_EXCEPTION_SCOPE_VERIFICATION 0
+#else
+#define ENABLE_EXCEPTION_SCOPE_VERIFICATION 1
#endif
+#endif
/* Pick which allocator to use; we only need an executable allocator if the assembler is compiled in.
On non-Windows x86-64, iOS, and ARM64 we use a single fixed mmap, on other platforms we mmap on demand. */
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
