[wasm] Generate source position information

src/compiler.h

Index: src/compiler.h
diff --git a/src/compiler.h b/src/compiler.h
index 88956f434fdc6c96aa0e199e97cb88a98b0b180a..b78dbee618ee5c2daf47bc3efdd2d2fe6e5cd96d 100644
--- a/src/compiler.h
+++ b/src/compiler.h
@@ -21,6 +21,9 @@ class JavaScriptFrame;
 class OptimizedCompileJob;
 class ParseInfo;
 class ScriptData;
+namespace compiler {
+class SourcePositionTable;
+}
 
 // The V8 compiler API.
 //
@@ -457,6 +460,14 @@ class CompilationInfo {
 
   StackFrame::Type GetOutputStackFrameType() const;
 
+  void SetSourcePositionTable(compiler::SourcePositionTable* table) {
+    source_position_table_ = table;

[titzer, 2016/04/18 11:03:51]

I'm not sure that the CompilationInfo is the best place to dump this, since it's
a compiler-internal field, but I don't have a better suggestion off the top of
my head.

[Michael Starzinger, 2016/04/18 11:08:47]

Not a fan of adding this to CompilationInfo either. IIUC, this is only needed to
pass the source position table from the WASM compiler into the TurboFan
pipeline. If we were to add a dedicated Pipeline::GenerateWASMCode, then we
could it pass in thorugh exactly that channel. This would also finally get rid
of the call to GenerateCodeForTesting from production code. WDYT?

[Clemens Hammacher, 2016/04/18 11:18:09]

Sounds reasonable. I will try to change it that way.

+  }
+
+  compiler::SourcePositionTable* GetSourcePositionTable() {
+    return source_position_table_;
+  }
+
  protected:
   ParseInfo* parse_info_;
 
@@ -554,6 +565,8 @@ class CompilationInfo {
 
   const char* debug_name_;
 
+  compiler::SourcePositionTable* source_position_table_ = nullptr;
+
   DISALLOW_COPY_AND_ASSIGN(CompilationInfo);
 };