Revision: 3298
Author: [email protected]
Date: Fri Nov 13 01:42:18 2009
Log: Cleanup and fix generated comments in the top-level compiler.

Review URL: http://codereview.chromium.org/384087
http://code.google.com/p/v8/source/detail?r=3298

Modified:
  /branches/bleeding_edge/src/arm/fast-codegen-arm.cc
  /branches/bleeding_edge/src/fast-codegen.cc
  /branches/bleeding_edge/src/ia32/fast-codegen-ia32.cc
  /branches/bleeding_edge/src/x64/fast-codegen-x64.cc

=======================================
--- /branches/bleeding_edge/src/arm/fast-codegen-arm.cc Thu Nov 12 03:38:01  
2009
+++ /branches/bleeding_edge/src/arm/fast-codegen-arm.cc Fri Nov 13 01:42:18  
2009
@@ -123,7 +123,7 @@
    // Put the lr setup instruction in the delay slot.  The kInstrSize is
    // added to the implicit 8 byte offset that always applies to operations
    // with pc and gives a return address 12 bytes down.
-  Comment cmnt(masm_, "[ Stack check");
+  { Comment cmnt(masm_, "[ Stack check");
    __ LoadRoot(r2, Heap::kStackLimitRootIndex);
    __ add(lr, pc, Operand(Assembler::kInstrSize));
    __ cmp(sp, Operand(r2));
@@ -133,6 +133,7 @@
                   RelocInfo::CODE_TARGET),
           LeaveCC,
           lo);
+  }

    { Comment cmnt(masm_, "[ Declarations");
      VisitDeclarations(fun->scope()->declarations());
@@ -326,6 +327,7 @@


  void FastCodeGenerator::VisitDeclaration(Declaration* decl) {
+  Comment cmnt(masm_, "[ Declaration");
    Variable* var = decl->proxy()->var();
    ASSERT(var != NULL);  // Must have been resolved.
    Slot* slot = var->slot();
=======================================
--- /branches/bleeding_edge/src/fast-codegen.cc Tue Nov 10 01:57:13 2009
+++ /branches/bleeding_edge/src/fast-codegen.cc Fri Nov 13 01:42:18 2009
@@ -76,7 +76,6 @@

  void FastCodeGenerator::VisitDeclarations(
      ZoneList<Declaration*>* declarations) {
-  Comment cmnt(masm_, "[ Declarations");
    int length = declarations->length();
    int globals = 0;
    for (int i = 0; i < length; i++) {
=======================================
--- /branches/bleeding_edge/src/ia32/fast-codegen-ia32.cc       Thu Nov 12  
03:38:01 2009
+++ /branches/bleeding_edge/src/ia32/fast-codegen-ia32.cc       Fri Nov 13  
01:42:18 2009
@@ -322,6 +322,7 @@


  void FastCodeGenerator::VisitDeclaration(Declaration* decl) {
+  Comment cmnt(masm_, "[ Declaration");
    Variable* var = decl->proxy()->var();
    ASSERT(var != NULL);  // Must have been resolved.
    Slot* slot = var->slot();
=======================================
--- /branches/bleeding_edge/src/x64/fast-codegen-x64.cc Thu Nov 12 03:38:01  
2009
+++ /branches/bleeding_edge/src/x64/fast-codegen-x64.cc Fri Nov 13 01:42:18  
2009
@@ -336,6 +336,7 @@


  void FastCodeGenerator::VisitDeclaration(Declaration* decl) {
+  Comment cmnt(masm_, "[ Declaration");
    Variable* var = decl->proxy()->var();
    ASSERT(var != NULL);  // Must have been resolved.
    Slot* slot = var->slot();

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to