Changeset 285712 in webkit

Timestamp:

Nov 12, 2021, 3:43:26 AM (5 years ago)

Author:

commit-queue@webkit.org

Message:

[JSC][32bit] Unskip JSTests/microbenchmarks/redefine-property-accessor-dictionary.js
​https://bugs.webkit.org/show_bug.cgi?id=233034

Unreviewed gardening.

This is skipped for memoryLimited *and* mips|arm systems, but a)
seems to run fine on both setups and b) is almost identical to the
other tests/benchmarks that landed in the same patch, that do not
have that limitation. Optimistically remove the skip and let's see
if it sticks.

Patch by Xan Lopez <Xan Lopez> on 2021-11-12

• microbenchmarks/redefine-property-accessor-dictionary.js:

Location:

trunk/JSTests

Files:

• ChangeLog (modified) (1 diff)
• microbenchmarks/redefine-property-accessor-dictionary.js (modified) (1 diff)

trunk/JSTests/ChangeLog

@@ +1 @@
+2021-11-12  Xan Lopez  <xan@igalia.com>
+
+        [JSC][32bit] Unskip JSTests/microbenchmarks/redefine-property-accessor-dictionary.js
+        https://bugs.webkit.org/show_bug.cgi?id=233034
+
+        Unreviewed gardening.
+
+        This is skipped for memoryLimited *and* mips|arm systems, but a)
+        seems to run fine on both setups and b) is almost identical to the
+        other tests/benchmarks that landed in the same patch, that do not
+        have that limitation. Optimistically remove the skip and let's see
+        if it sticks.
+
+        * microbenchmarks/redefine-property-accessor-dictionary.js:
+
 2021-11-11  Michael Saboff  <msaboff@apple.com>
 

trunk/JSTests/microbenchmarks/redefine-property-accessor-dictionary.js

@@ -1 @@
-//@ skip if $memoryLimited and ["arm", "mips"].include?($architecture)
 const obj = {};
 for (let i = 0; i < 100; ++i)