Reviewers: Mads Ager, Description: Tweak an assertion in the jump target destructor so that it only applies to initialized jump targets, and not in the case of a stack overflow.
Please review this at http://codereview.chromium.org/42118 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M src/jump-target.cc Index: src/jump-target.cc =================================================================== --- src/jump-target.cc (revision 1496) +++ src/jump-target.cc (working copy) @@ -70,7 +70,10 @@ void JumpTarget::Unuse() { - ASSERT(!is_linked()); + // We should not deallocate jump targets that have unresolved jumps + // to them. In the event of a compile-time stack overflow or an + // unitialized jump target, we don't care. + ASSERT(!is_linked() || cgen_ == NULL || cgen_->HasStackOverflow()); #ifdef DEBUG for (int i = 0; i < reaching_frames_.length(); i++) { ASSERT(reaching_frames_[i] == NULL); --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
