From 9c8f0f09ea6d6c08eeaf80019dd41da17627ee03 Mon Sep 17 00:00:00 2001 From: "Ilija.Pavlovic" Date: Fri, 2 Jun 2017 06:08:53 -0700 Subject: [PATCH] Fix tests regress-336820 and regress-678917. Execution tests regress-336820 and regress-678917 finished with error messages "allocation failure GC in old space requested" and "fatal javascript OOM in CALL_AND_RETRY_LAST". For successful execution, parameter max_old_space_size must be increased and this is done inside tests. TEST=mjsunit/regress/regress-336820, mjsunit/regress/regress-678917 BUG= Review-Url: https://codereview.chromium.org/2907363003 Cr-Original-Commit-Position: refs/heads/master@{#45672} Committed: https://chromium.googlesource.com/v8/v8/+/1550c869094c5873da9226ae9b4de7084692a857 Review-Url: https://codereview.chromium.org/2907363003 Cr-Commit-Position: refs/heads/master@{#45681} --- test/mjsunit/regress/regress-336820.js | 2 ++ test/mjsunit/regress/regress-678917.js | 2 ++ 2 files changed, 4 insertions(+) diff --git a/test/mjsunit/regress/regress-336820.js b/test/mjsunit/regress/regress-336820.js index 9b46629604..f305626596 100644 --- a/test/mjsunit/regress/regress-336820.js +++ b/test/mjsunit/regress/regress-336820.js @@ -25,6 +25,8 @@ // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. +// Flags: --max-old-space-size=400 + assertThrows((function() { s = "Hello World!\n"; while (true) { diff --git a/test/mjsunit/regress/regress-678917.js b/test/mjsunit/regress/regress-678917.js index accb515623..cb88ead5ba 100644 --- a/test/mjsunit/regress/regress-678917.js +++ b/test/mjsunit/regress/regress-678917.js @@ -2,6 +2,8 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. +// Flags: --max-old-space-size=400 + s1 = 'xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx'; s1 += s1; s1 += s1;