Skip to content

Commit b07a06c

Browse files
committed
tests: increase the timeout for SILOptimizer/large_nested_array.swift.gyb
The test run time jumped from ~200s to ~600s on Linux. Unfortunately I could not reproduce this locally. But what I saw is that ARCSequenceOpts is taking 75% of the compile time: rdar://144863155 Let's increase the timeout a bit until we can get rid of ARCSequenceOpts. rdar://144810758
1 parent 68fe078 commit b07a06c

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

validation-test/SILOptimizer/large_nested_array.swift.gyb

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,15 @@
22
// RUN: %gyb %s > %t/main.swift
33

44
// The compiler should finish in less than 2 minutes. To give some slack,
5-
// specify a timeout of 10 minutes.
6-
// If the compiler needs more than 10 minutes, there is probably a real problem.
5+
// specify a timeout of 12 minutes.
6+
7+
// TODO: 75% of compile time is spent in ARCSequenceOpts: rdar://144863155
8+
// The timeout should be decreased once we can get rid of ARCSequenceOpts
9+
10+
// If the compiler needs more than 12 minutes, there is probably a real problem.
711
// So please don't just increase the timeout in case this fails.
812

9-
// RUN: %{python} %S/../../test/Inputs/timeout.py 600 %target-swift-frontend -O -parse-as-library -sil-verify-none -c %t/main.swift -o %t/main.o
13+
// RUN: %{python} %S/../../test/Inputs/timeout.py 720 %target-swift-frontend -O -parse-as-library -sil-verify-none -c %t/main.swift -o %t/main.o
1014

1115
// REQUIRES: swift_stdlib_no_asserts,optimized_stdlib
1216
// REQUIRES: long_test

0 commit comments

Comments
 (0)