Refactor the compiling pipeline.

src/full-codegen.h

Index: src/full-codegen.h
diff --git a/src/full-codegen.h b/src/full-codegen.h
index 11d5341ecab48ee877a05f3ceb7b5d56a80cc287..35b864432758b03470c1f934fd96eadbecbf218c 100644
--- a/src/full-codegen.h
+++ b/src/full-codegen.h
@@ -928,9 +928,9 @@ class BackEdgeTable {
 
   // Change a back edge patched for on-stack replacement to perform a
   // stack check first.
-  static void AddStackCheck(CompilationInfo* info);
+  static void AddStackCheck(Code* code, uint32_t pc_offset);

[titzer, 2013/12/09 14:49:28]

Why not a Handle<Code>?

 
-  // Remove the stack check, if available, and replace by on-stack replacement.
+  // Revert the patch by AddStackCheck.
   static void RemoveStackCheck(CompilationInfo* info);

[titzer, 2013/12/09 14:49:28]

And why does this one take a CompilationInfo? Does it remove all checks, or just
one?

[Yang, 2013/12/10 11:22:04]

This removes only the one added in AddStackCheck. All the necessary information
are in the compilation info, so I though passing that would be cleaner.

[titzer, 2013/12/17 15:27:55]

I think Add and Remove should be symmetric, and neither should take a
CompilationInfo, which is an indirection to the stuff needed for the operation.

[Yang, 2013/12/17 16:31:33]

Made the signatures symmetrical.

 
   // Return the current patch state of the back edge.