89ca48c907
The arm implementation made the assumption that the {lhs} and {dst}
registers are either the same, or there is no overlap. This assumption
does not hold.
ia32 on the other hand has a lot of complicated logic (and unnecessary
code generation) for different cases of overlap.
This CL fixes the arm issue *and* simplifies the ia32 logic by making
the arm assumption hold, and using it to eliminate special handling on
ia32.
R=thibaudm@chromium.org
Bug: chromium:1146861
Change-Id: I8753c2ed70349e735c03293130c899c0c8a3a671
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2526388
Reviewed-by: Thibaud Michaud <thibaudm@chromium.org>
Commit-Queue: Clemens Backes <clemensb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#71060}
57 lines
1.8 KiB
JavaScript
57 lines
1.8 KiB
JavaScript
// Copyright 2020 the V8 project authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
// found in the LICENSE file.
|
|
|
|
load('test/mjsunit/wasm/wasm-module-builder.js');
|
|
|
|
const builder = new WasmModuleBuilder();
|
|
builder.addGlobal(kWasmI32, 1);
|
|
builder.addType(makeSig([], [kWasmF64]));
|
|
// Generate function 1 (out of 1).
|
|
builder.addFunction(undefined, 0 /* sig */)
|
|
.addLocals(kWasmI32, 8).addLocals(kWasmI64, 3)
|
|
.addBodyWithEnd([
|
|
// signature: d_v
|
|
// body:
|
|
kExprGlobalGet, 0x00, // global.get
|
|
kExprLocalSet, 0x00, // local.set
|
|
kExprI32Const, 0x00, // i32.const
|
|
kExprI32Eqz, // i32.eqz
|
|
kExprLocalSet, 0x01, // local.set
|
|
kExprGlobalGet, 0x00, // global.get
|
|
kExprLocalSet, 0x02, // local.set
|
|
kExprI32Const, 0x01, // i32.const
|
|
kExprI32Const, 0x01, // i32.const
|
|
kExprI32Sub, // i32.sub
|
|
kExprLocalSet, 0x03, // local.set
|
|
kExprGlobalGet, 0x00, // global.get
|
|
kExprLocalSet, 0x04, // local.set
|
|
kExprI32Const, 0x00, // i32.const
|
|
kExprI32Eqz, // i32.eqz
|
|
kExprLocalSet, 0x05, // local.set
|
|
kExprGlobalGet, 0x00, // global.get
|
|
kExprLocalSet, 0x06, // local.set
|
|
kExprI32Const, 0x00, // i32.const
|
|
kExprI32Const, 0x01, // i32.const
|
|
kExprI32Sub, // i32.sub
|
|
kExprLocalSet, 0x07, // local.set
|
|
kExprBlock, kWasmStmt, // block @45
|
|
kExprI32Const, 0x00, // i32.const
|
|
kExprIf, kWasmStmt, // if @49
|
|
kExprLocalGet, 0x0a, // local.get
|
|
kExprLocalSet, 0x08, // local.set
|
|
kExprElse, // else @55
|
|
kExprNop, // nop
|
|
kExprEnd, // end @57
|
|
kExprLocalGet, 0x08, // local.get
|
|
kExprLocalSet, 0x09, // local.set
|
|
kExprLocalGet, 0x09, // local.get
|
|
kExprI64Const, 0xff, 0x01, // i64.const
|
|
kExprI64Add, // i64.add
|
|
kExprDrop, // drop
|
|
kExprEnd, // end @69
|
|
kExprF64Const, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xf0, 0x3f, // f64.const
|
|
kExprEnd, // end @79
|
|
]);
|
|
builder.instantiate();
|