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

Side by Side Diff: src/compiler/wasm-compiler.h

Issue 2708593002: Revert of [wasm] Identify wasm functions with index into the function tables. (Closed)
Patch Set: Created 3 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
« no previous file with comments | « no previous file | src/compiler/wasm-compiler.cc » ('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 2015 the V8 project authors. All rights reserved. 1 // Copyright 2015 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef V8_COMPILER_WASM_COMPILER_H_ 5 #ifndef V8_COMPILER_WASM_COMPILER_H_
6 #define V8_COMPILER_WASM_COMPILER_H_ 6 #define V8_COMPILER_WASM_COMPILER_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 // Clients of this interface shouldn't depend on lots of compiler internals. 10 // Clients of this interface shouldn't depend on lots of compiler internals.
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 wasm::ModuleBytesEnv* module_env, 61 wasm::ModuleBytesEnv* module_env,
62 const wasm::WasmFunction* function) { 62 const wasm::WasmFunction* function) {
63 WasmCompilationUnit unit(thrower, isolate, module_env, function, 63 WasmCompilationUnit unit(thrower, isolate, module_env, function,
64 function->func_index); 64 function->func_index);
65 unit.ExecuteCompilation(); 65 unit.ExecuteCompilation();
66 return unit.FinishCompilation(); 66 return unit.FinishCompilation();
67 } 67 }
68 68
69 private: 69 private:
70 SourcePositionTable* BuildGraphForWasmFunction(double* decode_ms); 70 SourcePositionTable* BuildGraphForWasmFunction(double* decode_ms);
71 char* GetTaggedFunctionName(const wasm::WasmFunction* function);
72 71
73 wasm::ErrorThrower* thrower_; 72 wasm::ErrorThrower* thrower_;
74 Isolate* isolate_; 73 Isolate* isolate_;
75 wasm::ModuleBytesEnv* module_env_; 74 wasm::ModuleBytesEnv* module_env_;
76 const wasm::WasmFunction* function_; 75 const wasm::WasmFunction* function_;
77 // Function name is tagged with uint32 func_index - wasm#<func_index>
78 char function_name_[16];
79 // The graph zone is deallocated at the end of ExecuteCompilation. 76 // The graph zone is deallocated at the end of ExecuteCompilation.
80 std::unique_ptr<Zone> graph_zone_; 77 std::unique_ptr<Zone> graph_zone_;
81 JSGraph* jsgraph_; 78 JSGraph* jsgraph_;
82 Zone compilation_zone_; 79 Zone compilation_zone_;
83 CompilationInfo info_; 80 CompilationInfo info_;
84 std::unique_ptr<CompilationJob> job_; 81 std::unique_ptr<CompilationJob> job_;
85 uint32_t index_; 82 uint32_t index_;
86 wasm::Result<wasm::DecodeStruct*> graph_construction_result_; 83 wasm::Result<wasm::DecodeStruct*> graph_construction_result_;
87 bool ok_; 84 bool ok_;
88 ZoneVector<trap_handler::ProtectedInstructionData> 85 ZoneVector<trap_handler::ProtectedInstructionData>
(...skipping 299 matching lines...) Expand 10 before | Expand all | Expand 10 after
388 } 385 }
389 386
390 int AddParameterNodes(Node** args, int pos, int param_count, 387 int AddParameterNodes(Node** args, int pos, int param_count,
391 wasm::FunctionSig* sig); 388 wasm::FunctionSig* sig);
392 }; 389 };
393 } // namespace compiler 390 } // namespace compiler
394 } // namespace internal 391 } // namespace internal
395 } // namespace v8 392 } // namespace v8
396 393
397 #endif // V8_COMPILER_WASM_COMPILER_H_ 394 #endif // V8_COMPILER_WASM_COMPILER_H_
OLDNEW
« no previous file with comments | « no previous file | src/compiler/wasm-compiler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698