211313 2020-05-01 12:46:37 -0700 REGRESSION: stress/array-buffer-view-watchpoint-can-be-fired-in-really-add-in-dfg.js fails on ARM (thumb2 only?) 2020-05-08 14:08:12 -0700 1 1 1 Unclassified WebKit JavaScriptCore WebKit Nightly Build Unspecified Unspecified NEW https://bugs.webkit.org/show_bug.cgi?id=211304 https://bugs.webkit.org/show_bug.cgi?id=211205 https://bugs.webkit.org/show_bug.cgi?id=211020 P2 Normal --- 1 psaavedra webkit-unassigned ap ashvayka fpizlo keith_miller saam oldest_to_newest 1647987 0 psaavedra 2020-05-01 12:46:37 -0700 * Related to: https://bugs.webkit.org/show_bug.cgi?id=211304 * Regression appears after https://bugs.webkit.org/show_bug.cgi?id=211205 1648006 1 ashvayka 2020-05-01 13:03:29 -0700 I wonder if it is https://bugs.webkit.org/show_bug.cgi?id=211020 to blame for regression? The change for ArrayBuffer is barely observable though and the test was flaky before r260992. 1648232 2 psaavedra 2020-05-02 05:10:06 -0700 (In reply to Alexey Shvayka from comment #1) > I wonder if it is https://bugs.webkit.org/show_bug.cgi?id=211020 to blame > for regression? The change for ArrayBuffer is barely observable though and > the test was flaky before r260992. Yes, https://bugs.webkit.org/show_bug.cgi?id=211020 looks like a more porbable candidate. 1648233 3 psaavedra 2020-05-02 05:10:21 -0700 *probable 1648239 4 ashvayka 2020-05-02 07:06:52 -0700 Can I somehow obtain a crash log from EWS or reproduce the failure locally? 1649787 5 ap 2020-05-06 14:04:25 -0700 The test was skipped in https://trac.webkit.org/r261010. It was skipped for all ARM, which seems wrong, as we never had evidence of this affecting iOS. Index: /trunk/JSTests/stress/array-buffer-view-watchpoint-can-be-fired-in-really-add-in-dfg.js =================================================================== --- /trunk/JSTests/stress/array-buffer-view-watchpoint-can-be-fired-in-really-add-in-dfg.js (revision 261009) +++ /trunk/JSTests/stress/array-buffer-view-watchpoint-can-be-fired-in-really-add-in-dfg.js (revision 261010) @@ -1,2 +1,3 @@ +//@ skip if ["arm"].include?($architecture) //@ requireOptions("-e", "let iterations=40000") if ["mips"].include?($architecture) //@ runDefault("--jitPolicyScale=0") 1650556 6 psaavedra 2020-05-08 03:58:43 -0700 (In reply to Alexey Proskuryakov from comment #5) > The test was skipped in https://trac.webkit.org/r261010. It was skipped for > all ARM, which seems wrong, as we never had evidence of this affecting iOS. > > Index: > /trunk/JSTests/stress/array-buffer-view-watchpoint-can-be-fired-in-really- > add-in-dfg.js > =================================================================== > --- > /trunk/JSTests/stress/array-buffer-view-watchpoint-can-be-fired-in-really- > add-in-dfg.js (revision 261009) > +++ > /trunk/JSTests/stress/array-buffer-view-watchpoint-can-be-fired-in-really- > add-in-dfg.js (revision 261010) > @@ -1,2 +1,3 @@ > +//@ skip if ["arm"].include?($architecture) > //@ requireOptions("-e", "let iterations=40000") if > ["mips"].include?($architecture) > //@ runDefault("--jitPolicyScale=0") I think I rushed myself when I skipped the test with not enough information to fill the issue. I just re-added the test again. Let's see the results for the next builds. 1650756 7 psaavedra 2020-05-08 14:08:12 -0700 The test only is failing in JSCOnly Linux for ARMv7: * https://build.webkit.org/builders/JSCOnly%20Linux%20ARMv7%20Thumb2%20SoftFP%20Release/builds/5768 * https://build.webkit.org/builders/JSCOnly%20Linux%20ARMv7%20Thumb2%20Release/builds/11484