Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(24)

Side by Side Diff: src/hydrogen.cc

Issue 7618007: Simplify handling of exits from with and catch. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Incorporate review comments Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « src/full-codegen.cc ('k') | src/ia32/frames-ia32.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 the V8 project authors. All rights reserved. 1 // Copyright 2011 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 2616 matching lines...) Expand 10 before | Expand all | Expand 10 after
2627 ASSERT(context->IsValue()); 2627 ASSERT(context->IsValue());
2628 CHECK_ALIVE(VisitForValue(stmt->expression())); 2628 CHECK_ALIVE(VisitForValue(stmt->expression()));
2629 HValue* return_value = environment()->Pop(); 2629 HValue* return_value = environment()->Pop();
2630 current_block()->AddLeaveInlined(return_value, function_return()); 2630 current_block()->AddLeaveInlined(return_value, function_return());
2631 } 2631 }
2632 set_current_block(NULL); 2632 set_current_block(NULL);
2633 } 2633 }
2634 } 2634 }
2635 2635
2636 2636
2637 void HGraphBuilder::VisitEnterWithContextStatement( 2637 void HGraphBuilder::VisitWithStatement(WithStatement* stmt) {
2638 EnterWithContextStatement* stmt) {
2639 ASSERT(!HasStackOverflow()); 2638 ASSERT(!HasStackOverflow());
2640 ASSERT(current_block() != NULL); 2639 ASSERT(current_block() != NULL);
2641 ASSERT(current_block()->HasPredecessor()); 2640 ASSERT(current_block()->HasPredecessor());
2642 return Bailout("EnterWithContextStatement"); 2641 return Bailout("WithStatement");
2643 } 2642 }
2644 2643
2645 2644
2646 void HGraphBuilder::VisitExitContextStatement(ExitContextStatement* stmt) { 2645 void HGraphBuilder::VisitExitContextStatement(ExitContextStatement* stmt) {
2647 ASSERT(!HasStackOverflow()); 2646 ASSERT(!HasStackOverflow());
2648 ASSERT(current_block() != NULL); 2647 ASSERT(current_block() != NULL);
2649 ASSERT(current_block()->HasPredecessor()); 2648 ASSERT(current_block()->HasPredecessor());
2650 return Bailout("ExitContextStatement"); 2649 return Bailout("ExitContextStatement");
2651 } 2650 }
2652 2651
(...skipping 4097 matching lines...) Expand 10 before | Expand all | Expand 10 after
6750 } 6749 }
6751 } 6750 }
6752 6751
6753 #ifdef DEBUG 6752 #ifdef DEBUG
6754 if (graph_ != NULL) graph_->Verify(); 6753 if (graph_ != NULL) graph_->Verify();
6755 if (allocator_ != NULL) allocator_->Verify(); 6754 if (allocator_ != NULL) allocator_->Verify();
6756 #endif 6755 #endif
6757 } 6756 }
6758 6757
6759 } } // namespace v8::internal 6758 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/full-codegen.cc ('k') | src/ia32/frames-ia32.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698