Skip to content
Permalink
Browse files
[libpas] Enable libpas on macOS
https://bugs.webkit.org/show_bug.cgi?id=231815

Reviewed by Saam Barati.

Enabling libpas on x64 macOS. Previously, we enabled it only on AppleSilicon.
This helps stressing libpas on fuzzers more. And it offers large performance
improvement.

1. Speedometer2 is 2.1% improved on low-end macOS (MBA8,2) and 1.7% improved on high-end macOS (MBP14,1).
2. JetStream2 is neutral on both.

* bmalloc/BPlatform.h:

Canonical link: https://commits.webkit.org/243103@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@284305 268f45cc-cd09-0410-ab3c-d52691b4dbfc
  • Loading branch information
Constellation committed Oct 16, 2021
1 parent d498eb4 commit 9608c16a8f4ac579ce51b97e30f647c4c4024d96
Showing 2 changed files with 17 additions and 1 deletion.
@@ -1,3 +1,19 @@
2021-10-15 Yusuke Suzuki <ysuzuki@apple.com>

[libpas] Enable libpas on macOS
https://bugs.webkit.org/show_bug.cgi?id=231815

Reviewed by Saam Barati.

Enabling libpas on x64 macOS. Previously, we enabled it only on AppleSilicon.
This helps stressing libpas on fuzzers more. And it offers large performance
improvement.

1. Speedometer2 is 2.1% improved on low-end macOS (MBA8,2) and 1.7% improved on high-end macOS (MBP14,1).
2. JetStream2 is neutral on both.

* bmalloc/BPlatform.h:

2021-10-14 Yusuke Suzuki <ysuzuki@apple.com>

[libpas] Use `enum { ... }` instead of `static const size_t` for constant values used for array size
@@ -311,7 +311,7 @@

/* BENABLE(LIBPAS) is enabling libpas build. But this does not mean we use libpas for bmalloc replacement. */
#if !defined(BENABLE_LIBPAS)
#if BCPU(ARM64) && BOS(MAC)
#if BOS(MAC)
#define BENABLE_LIBPAS 1
#else
#define BENABLE_LIBPAS 0

0 comments on commit 9608c16

Please sign in to comment.