Reviewers: jarin,

Description:
[frames] No GC is allowed while using the unhandlified TranslatedState.

[email protected]

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

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

Affected files (+2, -0 lines):
  M src/frames.cc


Index: src/frames.cc
diff --git a/src/frames.cc b/src/frames.cc
index 644a0b3ec7391a6d5e7658b98e8bbec885eb1c8f..0b6942183990f92aa3676732b73ee09f2f0389cd 100644
--- a/src/frames.cc
+++ b/src/frames.cc
@@ -886,6 +886,7 @@ void OptimizedFrame::Summarize(List<FrameSummary>* frames) {

   // We create the summary in reverse order because the frames
   // in the deoptimization translation are ordered bottom-to-top.
+  DisallowHeapAllocation no_gc;
   TranslatedState state(this);
   bool is_constructor = IsConstructor();
   for (TranslatedFrame const& frame : state) {
@@ -987,6 +988,7 @@ void OptimizedFrame::GetFunctions(List<JSFunction*>* functions) {
     return JavaScriptFrame::GetFunctions(functions);
   }

+  DisallowHeapAllocation no_gc;
   TranslatedState state(this);
   for (TranslatedFrame const& frame : state) {
     if (frame.kind() == TranslatedFrame::kFunction) {


--
--
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