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

Side by Side Diff: src/hydrogen.cc

Issue 9373023: Extend AST with basic module constructs (yet unused). (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Fixed baseline. Created 8 years, 10 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
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 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 6564 matching lines...) Expand 10 before | Expand all | Expand 10 after
6575 environment()->Bind(var, value); 6575 environment()->Bind(var, value);
6576 } 6576 }
6577 } 6577 }
6578 break; 6578 break;
6579 case Variable::LOOKUP: 6579 case Variable::LOOKUP:
6580 return Bailout("unsupported lookup slot in declaration"); 6580 return Bailout("unsupported lookup slot in declaration");
6581 } 6581 }
6582 } 6582 }
6583 6583
6584 6584
6585 void HGraphBuilder::VisitModuleDeclaration(ModuleDeclaration* decl) {
6586 // TODO(rossberg)
6587 }
6588
6589
6590 void HGraphBuilder::VisitModuleLiteral(ModuleLiteral* module) {
6591 // TODO(rossberg)
6592 }
6593
Jakob Kummerow 2012/02/09 13:18:35 nit: we generally have two empty lines between top
rossberg 2012/02/09 13:26:31 Done.
6594 void HGraphBuilder::VisitModuleVariable(ModuleVariable* module) {
6595 // TODO(rossberg)
6596 }
6597
6598 void HGraphBuilder::VisitModulePath(ModulePath* module) {
6599 // TODO(rossberg)
6600 }
6601
6602 void HGraphBuilder::VisitModuleUrl(ModuleUrl* module) {
6603 // TODO(rossberg)
6604 }
6605
6606
6585 // Generators for inline runtime functions. 6607 // Generators for inline runtime functions.
6586 // Support for types. 6608 // Support for types.
6587 void HGraphBuilder::GenerateIsSmi(CallRuntime* call) { 6609 void HGraphBuilder::GenerateIsSmi(CallRuntime* call) {
6588 ASSERT(call->arguments()->length() == 1); 6610 ASSERT(call->arguments()->length() == 1);
6589 CHECK_ALIVE(VisitForValue(call->arguments()->at(0))); 6611 CHECK_ALIVE(VisitForValue(call->arguments()->at(0)));
6590 HValue* value = Pop(); 6612 HValue* value = Pop();
6591 HIsSmiAndBranch* result = new(zone()) HIsSmiAndBranch(value); 6613 HIsSmiAndBranch* result = new(zone()) HIsSmiAndBranch(value);
6592 return ast_context()->ReturnControl(result, call->id()); 6614 return ast_context()->ReturnControl(result, call->id());
6593 } 6615 }
6594 6616
(...skipping 1033 matching lines...) Expand 10 before | Expand all | Expand 10 after
7628 } 7650 }
7629 } 7651 }
7630 7652
7631 #ifdef DEBUG 7653 #ifdef DEBUG
7632 if (graph_ != NULL) graph_->Verify(false); // No full verify. 7654 if (graph_ != NULL) graph_->Verify(false); // No full verify.
7633 if (allocator_ != NULL) allocator_->Verify(); 7655 if (allocator_ != NULL) allocator_->Verify();
7634 #endif 7656 #endif
7635 } 7657 }
7636 7658
7637 } } // namespace v8::internal 7659 } } // namespace v8::internal
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698