commit 29c549bfe2ef50ff2ac322c75db5c7c55e943fb0
author Samir Jindel <sjindel@google.com> Wed Jun 27 14:52:47 2018 +0000
committer commit-bot@chromium.org <commit-bot@chromium.org> Wed Jun 27 14:52:47 2018 +0000
tree f956fdd9984622dc228c9a564b15b96cf85f0f32
parent e13b9e8a75b2beb6ef91e5b7dbc67971e7bc84d9

[vm/kernel] Use old space in a specific Kernel FE error allocation.

Test Plan:

This patches over #33648 for me.
It appears the fixed code is not tested already, because Object::Allocate
asserts that only the mutator thread issues new-space allocations.

There appears to still be an underlying issue which will be addressed
separately.

Change-Id: I1550a71e84ccd66bd6ddd56bfe5db1feb5a815bc
Reviewed-on: https://dart-review.googlesource.com/62501
Reviewed-by: Vyacheslav Egorov <vegorov@google.com>
Commit-Queue: Samir Jindel <sjindel@google.com>

runtime/vm/object.cc

diff --git a/runtime/vm/object.cc b/runtime/vm/object.cc
index e31beec..2440091 100644
--- a/runtime/vm/object.cc
+++ b/runtime/vm/object.cc
@@ -18654,7 +18654,7 @@
       // meaningless, therefore use the token index of this type parameter.
       *bound_error = LanguageError::NewFormatted(
           *bound_error, script, token_pos(), Report::AtLocation,
-          Report::kMalboundedType, Heap::kNew,
+          Report::kMalboundedType, Heap::kOld,
           "type parameter '%s' of class '%s' must extend bound '%s', "
           "but type argument '%s' is not a subtype of '%s'",
           type_param_name.ToCString(), class_name.ToCString(),