Reviewers: Michael Achenbach,

Message:
I'll check whether this the same reason why it's disabled for TF runs. If so,
I'll remove that line as well.

Description:
Disable webkit/array-iterate-backwards for debug builds.

(Debug builds have --enable-slow-asserts, and with slow asserts each
assignment to the array in the prepare function will trigger a check
over the entire array. This turns it into n^2 work, for reasonably
large n.)

BUG=

Please review this at https://codereview.chromium.org/831953003/

Base URL: https://chromium.googlesource.com/v8/v8.git@master

Affected files (+2, -4 lines):
  M test/webkit/webkit.status


Index: test/webkit/webkit.status
diff --git a/test/webkit/webkit.status b/test/webkit/webkit.status
index c33f1b928e7c208f1c260f3cae1136e0df338cde..fb8d77d8d2069abdb90594d7bef57155f7e4f0a5 100644
--- a/test/webkit/webkit.status
+++ b/test/webkit/webkit.status
@@ -44,9 +44,8 @@
   'dfg-double-vote-fuzz': [SKIP],
   'reentrant-caching': [SKIP],
   'sort-large-array': [SKIP],
-  # Too slow on windows with --nocrankshaft.
-  # TODO(mstarzinger): Too slow with TF.
-  'array-iterate-backwards': [PASS, NO_VARIANTS],
+  # Too slow with --enable-slow-asserts.
+  'array-iterate-backwards': [SKIP],
 }],  # 'mode == debug'
 ['simulator', {
   # Skip tests that timeout with turbofan.
@@ -71,7 +70,6 @@

 ['gc_stress == True and mode == debug', {
   # Skip tests that timeout.
-  'array-iterate-backwards': [SKIP]
 }],  # 'gc_stress == True and mode == debug'

##############################################################################


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to