Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package webkit2gtk3 for openSUSE:Factory 
checked in at 2024-09-09 14:43:13
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/webkit2gtk3 (Old)
 and      /work/SRC/openSUSE:Factory/.webkit2gtk3.new.10096 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "webkit2gtk3"

Mon Sep  9 14:43:13 2024 rev:200 rq:1199439 version:2.44.3

Changes:
--------
--- /work/SRC/openSUSE:Factory/webkit2gtk3/webkit2gtk3.changes  2024-08-19 
23:45:30.354143002 +0200
+++ /work/SRC/openSUSE:Factory/.webkit2gtk3.new.10096/webkit2gtk3.changes       
2024-09-09 14:43:17.587842759 +0200
@@ -1,0 +2,6 @@
+Sun Sep  1 16:30:22 UTC 2024 - Bjørn Lie <bjorn....@gmail.com>
+
+- Add revert-271175.patch: This reverts commit 279c9d7, it broke
+  wasm.
+
+-------------------------------------------------------------------

New:
----
  revert-271175.patch

BETA DEBUG BEGIN:
  New:
- Add revert-271175.patch: This reverts commit 279c9d7, it broke
  wasm.
BETA DEBUG END:

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ webkit2gtk3.spec ++++++
--- /var/tmp/diff_new_pack.T9ygud/_old  2024-09-09 14:43:18.271871141 +0200
+++ /var/tmp/diff_new_pack.T9ygud/_new  2024-09-09 14:43:18.275871307 +0200
@@ -91,6 +91,8 @@
 Patch0:         reproducibility.patch
 # PATCH-FIX-UPSTREAM webkit2gtk3-disable-dmabuf-nvidia.patch boo#1216778 
mgo...@suse.com -- disable the DMABuf renderer for NVIDIA proprietary drivers.
 Patch2:         webkit2gtk3-disable-dmabuf-nvidia.patch
+# PATCH-FIX-UPSTREAM revert-271175.patch -- This reverts commit 279c9d7, it 
broke wasm.
+Patch3:         revert-271175.patch
 
 BuildRequires:  Mesa-libEGL-devel
 BuildRequires:  Mesa-libGL-devel

++++++ revert-271175.patch ++++++
>From 9140ce712aa87091613874d802787ab476be0e39 Mon Sep 17 00:00:00 2001
From: Michael Catanzaro <mcatanz...@redhat.com>
Date: Wed, 14 Aug 2024 14:58:05 -0500
Subject: [PATCH] Revert "Cherry-pick 272448.770@safari-7618-branch
 (6d311cd7fefc). https://bugs.webkit.org/show_bug.cgi?id=271175";
 https://bugs.webkit.org/show_bug.cgi?id=278113

This reverts commit 279c9d7963182cc35cf4e0bfebe87df2d83eaef8.

This broke wasm, and I don't know how to fix it.

Canonical link: https://commits.webkit.org/274313.373@webkitglib/2.44
---
 .../stress/many-calls-results-on-stack.js     | 39 -------------------
 Source/JavaScriptCore/wasm/WasmBBQJIT.cpp     | 19 ---------
 2 files changed, 58 deletions(-)
 delete mode 100644 JSTests/wasm/stress/many-calls-results-on-stack.js

index 9049865e8ce7..3f142cf5e90f 100644
--- a/Source/JavaScriptCore/wasm/WasmBBQJIT.cpp
+++ b/Source/JavaScriptCore/wasm/WasmBBQJIT.cpp
@@ -3958,25 +3958,6 @@ void BBQJIT::returnValuesFromCall(Vector<Value, N>& 
results, const FunctionSigna
                     ASSERT(m_validFPRs.contains(returnLocation.asFPR(), 
Width::Width128));
                     m_fprSet.add(returnLocation.asFPR(), Width::Width128);
                 }
-            } else {
-                ASSERT(returnLocation.isStackArgument());
-                // FIXME: Ideally, we would leave these values where they are 
but a subsequent call could clobber them before they are used.
-                // That said, stack results are very rare so this isn't too 
painful.
-                // Even if we did leave them where they are, we'd need to 
flush them to their canonical location at the next branch otherwise
-                // we could have something like (assume no result regs for 
simplicity):
-                // call (result i32 i32) $foo
-                // if (result i32) // Stack: i32(StackArgument:8) 
i32(StackArgument:0)
-                //   // Stack: i32(StackArgument:8)
-                // else
-                //   call (result i32 i32) $bar // Stack: i32(StackArgument:8) 
we have to flush the stack argument to make room for the result of bar
-                //   drop // Stack: i32(Stack:X) i32(StackArgument:8) 
i32(StackArgument:0)
-                //   drop // Stack: i32(Stack:X) i32(StackArgument:8)
-                // end
-                // return // Stack i32(*Conflicting locations*)
-
-                Location canonicalLocation = canonicalSlot(result);
-                emitMoveMemory(result.type(), returnLocation, 
canonicalLocation);
-                returnLocation = canonicalLocation;
             }
         }
         bind(result, returnLocation);

Reply via email to