[webkit-changes] [233541] tags/Safari-606.1.24

Thu, 05 Jul 2018 14:43:49 -0700

Title: [233541] tags/Safari-606.1.24

Diff

Modified: tags/Safari-606.1.24/LayoutTests/ChangeLog (233540 => 233541)


--- tags/Safari-606.1.24/LayoutTests/ChangeLog	2018-07-05 21:35:09 UTC (rev 233540)
+++ tags/Safari-606.1.24/LayoutTests/ChangeLog	2018-07-05 21:43:35 UTC (rev 233541)
@@ -1,3 +1,7 @@
+2018-07-05  Kocsen Chung  <[email protected]>
+
+        Revert r233417. rdar://problem/41863878
+
 2018-07-04  Carlos Garcia Campos  <[email protected]>
 
         REGRESSION(r233325): [GTK] Broke 40 animations tests

Modified: tags/Safari-606.1.24/Source/WTF/ChangeLog (233540 => 233541)


--- tags/Safari-606.1.24/Source/WTF/ChangeLog	2018-07-05 21:35:09 UTC (rev 233540)
+++ tags/Safari-606.1.24/Source/WTF/ChangeLog	2018-07-05 21:43:35 UTC (rev 233541)
@@ -1,3 +1,7 @@
+2018-07-05  Kocsen Chung  <[email protected]>
+
+        Revert r233417. rdar://problem/41863878
+
 2018-07-04  Guillaume Emont  <[email protected]>
 
         FunctionTraits: Make cCallArity() constant on 32-bits.

Modified: tags/Safari-606.1.24/Source/WTF/wtf/Assertions.h (233540 => 233541)


--- tags/Safari-606.1.24/Source/WTF/wtf/Assertions.h	2018-07-05 21:35:09 UTC (rev 233540)
+++ tags/Safari-606.1.24/Source/WTF/wtf/Assertions.h	2018-07-05 21:43:35 UTC (rev 233541)
@@ -220,12 +220,6 @@
 #define WTFBreakpointTrap() WTFCrash() // Not implemented.
 #endif
 
-#if COMPILER(MSVC)
-#define WTFBreakpointTrapUnderConstexprContext() __debugbreak()
-#else
-#define WTFBreakpointTrapUnderConstexprContext() __builtin_trap()
-#endif
-
 #ifndef CRASH
 
 #if defined(NDEBUG) && OS(DARWIN)
@@ -236,13 +230,8 @@
     WTFBreakpointTrap(); \
     __builtin_unreachable(); \
 } while (0)
-#define CRASH_UNDER_CONSTEXPR_CONTEXT() do { \
-    WTFBreakpointTrapUnderConstexprContext(); \
-    __builtin_unreachable(); \
-} while (0)
 #else
 #define CRASH() WTFCrash()
-#define CRASH_UNDER_CONSTEXPR_CONTEXT() WTFCrash()
 #endif
 
 #endif // !defined(CRASH)
@@ -516,25 +505,6 @@
 #define RELEASE_ASSERT_NOT_REACHED() ASSERT_NOT_REACHED()
 #endif
 
-/* RELEASE_ASSERT_UNDER_CONSTEXPR_CONTEXT
-
-   This is a special version of RELEASE_ASSERT(assertion) that can be used in constexpr contexts.
-*/
-#if ASSERT_DISABLED
-#define RELEASE_ASSERT_UNDER_CONSTEXPR_CONTEXT(assertion) do { \
-    if (UNLIKELY(!(assertion))) { \
-        CRASH_UNDER_CONSTEXPR_CONTEXT(); \
-    } \
-} while (0)
-#else
-#define RELEASE_ASSERT_UNDER_CONSTEXPR_CONTEXT(assertion) do { \
-    if (!(assertion)) { \
-        WTFReportAssertionFailure(__FILE__, __LINE__, WTF_PRETTY_FUNCTION, #assertion); \
-        CRASH_UNDER_CONSTEXPR_CONTEXT(); \
-    } \
-} while (0)
-#endif
-
 /* UNREACHABLE_FOR_PLATFORM */
 
 #if COMPILER(CLANG)

Modified: tags/Safari-606.1.24/Source/WTF/wtf/Optional.h (233540 => 233541)


--- tags/Safari-606.1.24/Source/WTF/wtf/Optional.h	2018-07-05 21:35:09 UTC (rev 233540)
+++ tags/Safari-606.1.24/Source/WTF/wtf/Optional.h	2018-07-05 21:43:35 UTC (rev 233541)
@@ -530,7 +530,8 @@
   }
 
   OPTIONAL_MUTABLE_CONSTEXPR T* operator ->() {
-    RELEASE_ASSERT_UNDER_CONSTEXPR_CONTEXT(initialized());
+    // FIXME: We need to offer special assert function that can be used under the contexpr context.
+    // CONSTEXPR_ASSERT(initialized());
     return dataptr();
   }
 
@@ -539,27 +540,32 @@
   }
 
   OPTIONAL_MUTABLE_CONSTEXPR T& operator *() & {
-    RELEASE_ASSERT_UNDER_CONSTEXPR_CONTEXT(initialized());
+    // FIXME: We need to offer special assert function that can be used under the contexpr context.
+    // CONSTEXPR_ASSERT(initialized());
     return contained_val();
   }
 
   OPTIONAL_MUTABLE_CONSTEXPR T&& operator *() && {
-    RELEASE_ASSERT_UNDER_CONSTEXPR_CONTEXT(initialized());
+    // FIXME: We need to offer special assert function that can be used under the contexpr context.
+    // CONSTEXPR_ASSERT(initialized());
     return detail_::constexpr_move(contained_val());
   }
 
   constexpr T const& value() const& {
-    RELEASE_ASSERT_UNDER_CONSTEXPR_CONTEXT(initialized());
+    // FIXME: We need to offer special assert function that can be used under the contexpr context.
+    // return initialized() ? contained_val() : (throw bad_optional_access("bad optional access"), contained_val());
     return contained_val();
   }
 
   OPTIONAL_MUTABLE_CONSTEXPR T& value() & {
-    RELEASE_ASSERT_UNDER_CONSTEXPR_CONTEXT(initialized());
+    // FIXME: We need to offer special assert function that can be used under the contexpr context.
+    // return initialized() ? contained_val() : (throw bad_optional_access("bad optional access"), contained_val());
     return contained_val();
   }
 
   OPTIONAL_MUTABLE_CONSTEXPR T&& value() && {
-    RELEASE_ASSERT_UNDER_CONSTEXPR_CONTEXT(initialized());
+    // FIXME: We need to offer special assert function that can be used under the contexpr context.
+    // if (!initialized()) __THROW_EXCEPTION(bad_optional_access("bad optional access"));
     return std::move(contained_val());
   }
 
@@ -677,7 +683,8 @@
   }
 
   constexpr T& value() const {
-    RELEASE_ASSERT_UNDER_CONSTEXPR_CONTEXT(ref());
+    // FIXME: We need to offer special assert function that can be used under the contexpr context.
+    // return ref ? *ref : (throw bad_optional_access("bad optional access"), *ref);
     return *ref;
   }
 

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to