commit 04e307bd7735901457db31d96e7e881519f04d8e
author commit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc> Tue Oct 15 16:35:05 2019 +0000
committer commit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc> Tue Oct 15 16:35:05 2019 +0000
tree 3893a16e155e89fa5e55c1c5ad3bbb6444973737
parent 9c67cd80f9fb5e3bf0f07d348f1d17932ddab816

Interpreter: Don't assert that reference is nonnull
https://bugs.webkit.org/show_bug.cgi?id=202986

Patch by Angelos Oikonomopoulos <aoikonomopoulos@igalia.com> on 2019-10-15
Reviewed by Keith Miller.

G++ 9.2 can assume that the address of a reference is nonnull and
emits multiple warnings to that effect in --debug builds.

* interpreter/FrameTracers.h:
(JSC::NativeCallFrameTracer::NativeCallFrameTracer):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@251139 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog

diff --git a/Source/JavaScriptCore/ChangeLog b/Source/JavaScriptCore/ChangeLog
index 37531d5..fc5e472 100644
--- a/Source/JavaScriptCore/ChangeLog
+++ b/Source/JavaScriptCore/ChangeLog
@@ -1,3 +1,16 @@
+2019-10-15  Angelos Oikonomopoulos  <aoikonomopoulos@igalia.com>
+
+        Interpreter: Don't assert that reference is nonnull
+        https://bugs.webkit.org/show_bug.cgi?id=202986
+
+        Reviewed by Keith Miller.
+
+        G++ 9.2 can assume that the address of a reference is nonnull and
+        emits multiple warnings to that effect in --debug builds.
+
+        * interpreter/FrameTracers.h:
+        (JSC::NativeCallFrameTracer::NativeCallFrameTracer):
+
 2019-10-14  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r251090.

Source/JavaScriptCore/interpreter/FrameTracers.h

diff --git a/Source/JavaScriptCore/interpreter/FrameTracers.h b/Source/JavaScriptCore/interpreter/FrameTracers.h
index 1de8e4a..11f7b8a 100644
--- a/Source/JavaScriptCore/interpreter/FrameTracers.h
+++ b/Source/JavaScriptCore/interpreter/FrameTracers.h
@@ -85,7 +85,6 @@
 public:
     ALWAYS_INLINE NativeCallFrameTracer(VM& vm, CallFrame* callFrame)
     {
-        ASSERT(&vm);
         ASSERT(callFrame);
         ASSERT(reinterpret_cast<void*>(callFrame) < reinterpret_cast<void*>(vm.topEntryFrame));
         assertStackPointerIsAligned();