[arch-commits] Commit in electron11/trunk (PKGBUILD enable-dse-memoryssa.patch)

[Evangelos Foutras]

    Date: Monday, November 22, 2021 @ 11:31:16
  Author: foutrelis
Revision: 1054238

upgpkg: electron11 11.5.0-3: icu 70.1 rebuild

Added:
  electron11/trunk/enable-dse-memoryssa.patch
Modified:
  electron11/trunk/PKGBUILD

----------------------------+
 PKGBUILD                   |    5 +++-
 enable-dse-memoryssa.patch |   50 +++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 54 insertions(+), 1 deletion(-)

Modified: PKGBUILD
===================================================================
--- PKGBUILD	2021-11-22 10:47:02 UTC (rev 1054237)
+++ PKGBUILD	2021-11-22 11:31:16 UTC (rev 1054238)
@@ -4,7 +4,7 @@
 pkgver=11.5.0
 _commit=3d0705d81fd8b829372a732577b5c275352a9da6
 _chromiumver=87.0.4280.141
-pkgrel=2
+pkgrel=3
 pkgdesc='Build cross platform desktop apps with web technologies'
 arch=('x86_64')
 url='https://electronjs.org/'
@@ -29,6 +29,7 @@
         'chromium-glibc-2.33.patch'
         'chromium-harfbuzz-3.0.0.patch'
         'skia-harfbuzz-3.0.0.patch'
+        'enable-dse-memoryssa.patch'
         'chromium-include-limits.patch'
         'chromium-include-stddef.patch'
         'chromium-skia-harmony.patch'
@@ -46,6 +47,7 @@
             '2fccecdcd4509d4c36af873988ca9dbcba7fdb95122894a9fdf502c33a1d7a4b'
             '7ce947944a139e66774dfc7249bf7c3069f07f83a0f1b2c1a1b14287a7e15928'
             'c9ed1dbadaf4be6097e25bc5577b91751799befc2d0376b143e1bd10def5754e'
+            '0760a499534588674a173605f6bad1336e9a35ef97209cb0fed0f3573ce170cf'
             'de7090b3f449372dd721b64eb10a02d15a1018961def0b4f35d2c90167ded524'
             '2c76f4b2572ec9f065433199e0cfaf58a5cfc869920c6c851150e9bcfe4d6d2a'
             '771292942c0901092a402cc60ee883877a99fb804cb54d568c8c6c94565a48e1'
@@ -144,6 +146,7 @@
   patch -Np0 -i ../chromium-skia-harmony.patch
   patch -Np1 -i ../chromium-glibc-2.33.patch
   patch -Np1 -i ../chromium-harfbuzz-3.0.0.patch
+  patch -Np1 -i ../enable-dse-memoryssa.patch
   patch -Np1 -d third_party/skia <../skia-harfbuzz-3.0.0.patch
   patch -Np1 -i ../chromium-include-limits.patch
   patch -Np1 -i ../chromium-include-stddef.patch

Added: enable-dse-memoryssa.patch
===================================================================
--- enable-dse-memoryssa.patch	                        (rev 0)
+++ enable-dse-memoryssa.patch	2021-11-22 11:31:16 UTC (rev 1054238)
@@ -0,0 +1,50 @@
+From 4f59b0a437416e001ba96be22dfcd39806af70e6 Mon Sep 17 00:00:00 2001
+From: Nico Weber <thakis at chromium.org>
+Date: Fri, 6 Nov 2020 18:50:33 +0000
+Subject: [PATCH] build: Enable MemorySSA DSE with pinned clang.
+
+The feature has been on on the tot bots for a while now and it's been
+on by default upstream for a while, and things seem to work.
+
+Since we just rolled in a clang with all the fixes, let's turn this on.
+
+Bug: 1127713
+Change-Id: I56805d576bea5936d58c653da994dc738c2a79a3
+Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2520013
+Commit-Queue: Nico Weber <thakis at chromium.org>
+Reviewed-by: Hans Wennborg <hans at chromium.org>
+Cr-Commit-Position: refs/heads/master@{#824947}
+---
+ build/config/compiler/BUILD.gn | 19 -------------------
+ 1 file changed, 19 deletions(-)
+
+diff --git a/build/config/compiler/BUILD.gn b/build/config/compiler/BUILD.gn
+index 6f8c779040251..52912b1cb546a 100644
+--- a/build/config/compiler/BUILD.gn
++++ b/build/config/compiler/BUILD.gn
+@@ -490,25 +490,6 @@ config("compiler") {
+       "-mllvm",
+       "-instcombine-lower-dbg-declare=0",
+     ]
+-
+-    # Pinned clang has enable-dse-memoryssa enabled by default but it's broken,
+-    # so we have to force it off.
+-    # Trunk clang has it disabled by default but it might work, so we force it
+-    # on so the ToT bots can check if it works now.
+-    if (!llvm_force_head_revision) {
+-      cflags += [
+-        # TODO(https://crbug.com/1127713): Investigate, remove.
+-        "-mllvm",
+-        "-enable-dse-memoryssa=false",
+-      ]
+-    }
+-    if (llvm_force_head_revision) {
+-      cflags += [
+-        # TODO(https://crbug.com/1127713): Investigate, remove.
+-        "-mllvm",
+-        "-enable-dse-memoryssa=true",
+-      ]
+-    }
+   }
+ 
+   # C11/C++11 compiler flags setup.