Title: [187167] trunk/Source/_javascript_Core
Revision
187167
Author
o...@webkit.org
Date
2015-07-22 11:04:12 -0700 (Wed, 22 Jul 2015)

Log Message

Enable STATIC_ASSERT_IS_TRIVIALLY_DESTRUCTIBLE for GCC
https://bugs.webkit.org/show_bug.cgi?id=146829

Reviewed by Brent Fulgham.

* heap/GCAssertions.h:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (187166 => 187167)


--- trunk/Source/_javascript_Core/ChangeLog	2015-07-22 17:56:22 UTC (rev 187166)
+++ trunk/Source/_javascript_Core/ChangeLog	2015-07-22 18:04:12 UTC (rev 187167)
@@ -1,3 +1,12 @@
+2015-07-22  Csaba Osztrogonác  <o...@webkit.org>
+
+        Enable STATIC_ASSERT_IS_TRIVIALLY_DESTRUCTIBLE for GCC
+        https://bugs.webkit.org/show_bug.cgi?id=146829
+
+        Reviewed by Brent Fulgham.
+
+        * heap/GCAssertions.h:
+
 2015-07-22  Alex Christensen  <achristen...@webkit.org>
 
         Fix quirks in CMake build on Mac and Windows

Modified: trunk/Source/_javascript_Core/heap/GCAssertions.h (187166 => 187167)


--- trunk/Source/_javascript_Core/heap/GCAssertions.h	2015-07-22 17:56:22 UTC (rev 187166)
+++ trunk/Source/_javascript_Core/heap/GCAssertions.h	2015-07-22 18:04:12 UTC (rev 187167)
@@ -45,14 +45,6 @@
 #define ASSERT_GC_OBJECT_INHERITS(object, classInfo) do { (void)object; (void)classInfo; } while (0)
 #endif
 
-#if COMPILER(CLANG)
 #define STATIC_ASSERT_IS_TRIVIALLY_DESTRUCTIBLE(klass) static_assert(std::is_trivially_destructible<klass>::value, #klass " must have a trivial destructor")
-#elif COMPILER(MSVC)
-// An earlier verison of the C++11 spec used to call this type trait std::has_trivial_destructor, and that's what MSVC uses.
-#define STATIC_ASSERT_IS_TRIVIALLY_DESTRUCTIBLE(klass) static_assert(std::has_trivial_destructor<klass>::value, #klass " must have a trivial destructor")
-#else
-// This is not enabled on GCC due to http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52702
-#define STATIC_ASSERT_IS_TRIVIALLY_DESTRUCTIBLE(klass)
-#endif
 
 #endif // GCAssertions_h
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to