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

Unified Diff: src/compiler/wasm-compiler.cc

Issue 2764583003: [wasm] Fix wasm interpreter entry for 32 bit big endian systems (Closed)
Patch Set: Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler/wasm-compiler.cc
diff --git a/src/compiler/wasm-compiler.cc b/src/compiler/wasm-compiler.cc
index 57a77608f9687f9a34dc6aef05413dcb15ef51ab..5e67fc62eff4404756fff9e72f0174d7fdc624a6 100644
--- a/src/compiler/wasm-compiler.cc
+++ b/src/compiler/wasm-compiler.cc
@@ -2849,7 +2849,11 @@ void WasmGraphBuilder::BuildWasmInterpreterEntry(
graph()->NewNode(jsgraph()->machine()->Load(load_rep), arg_buffer,
Int32Constant(sizeof(int32_t)), lower, *control_);
*effect_ = upper;
+#if defined(V8_TARGET_LITTLE_ENDIAN)
Clemens Hammacher 2017/03/21 08:36:10 Instead of introducing another "#if defined(...)",
rayb 2017/03/22 17:21:47 Done.
Return(lower, upper);
+#elif defined(V8_TARGET_BIG_ENDIAN)
+ Return(upper, lower);
+#endif
} else {
MachineType load_rep = wasm::WasmOpcodes::MachineTypeFor(sig->GetReturn());
Node* val =
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698