2016-07-15 12:03:04 +00:00
|
|
|
// Copyright 2015 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.
|
|
|
|
|
|
|
|
#include <vector>
|
|
|
|
|
|
|
|
#include "src/v8.h"
|
|
|
|
|
|
|
|
#include "src/interpreter/bytecode-decoder.h"
|
2017-02-23 14:25:45 +00:00
|
|
|
#include "src/runtime/runtime.h"
|
2016-07-15 12:03:04 +00:00
|
|
|
#include "test/unittests/interpreter/bytecode-utils.h"
|
|
|
|
#include "test/unittests/test-utils.h"
|
|
|
|
|
|
|
|
namespace v8 {
|
|
|
|
namespace internal {
|
|
|
|
namespace interpreter {
|
|
|
|
|
|
|
|
TEST(BytecodeDecoder, DecodeBytecodeAndOperands) {
|
|
|
|
struct BytecodesAndResult {
|
|
|
|
const uint8_t bytecode[32];
|
|
|
|
const size_t length;
|
|
|
|
int parameter_count;
|
|
|
|
const char* output;
|
|
|
|
};
|
|
|
|
|
|
|
|
const BytecodesAndResult cases[] = {
|
|
|
|
{{B(LdaSmi), U8(1)}, 2, 0, " LdaSmi [1]"},
|
|
|
|
{{B(Wide), B(LdaSmi), U16(1000)}, 4, 0, " LdaSmi.Wide [1000]"},
|
|
|
|
{{B(ExtraWide), B(LdaSmi), U32(100000)},
|
|
|
|
6,
|
|
|
|
0,
|
|
|
|
"LdaSmi.ExtraWide [100000]"},
|
|
|
|
{{B(LdaSmi), U8(-1)}, 2, 0, " LdaSmi [-1]"},
|
|
|
|
{{B(Wide), B(LdaSmi), U16(-1000)}, 4, 0, " LdaSmi.Wide [-1000]"},
|
|
|
|
{{B(ExtraWide), B(LdaSmi), U32(-100000)},
|
|
|
|
6,
|
|
|
|
0,
|
|
|
|
"LdaSmi.ExtraWide [-100000]"},
|
|
|
|
{{B(Star), R8(5)}, 2, 0, " Star r5"},
|
|
|
|
{{B(Wide), B(Star), R16(136)}, 4, 0, " Star.Wide r136"},
|
Revert "[ignition] Add call bytecodes for undefined receiver"
This reverts commit 751e89359123df4f6e5f0b28b89fcdf63ed16a77.
Reason for revert: Breaks layout tests:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/14885
See:
https://github.com/v8/v8/wiki/Blink-layout-tests
Original change's description:
> [ignition] Add call bytecodes for undefined receiver
>
> Adds a collection of call bytecodes which have an implicit undefined
> receiver argument, for cases such as global calls where we know that the
> receiver has to be undefined. This way we can skip an LdaUndefined,
> decrease bytecode register pressure, and set a more accurate
> ConvertReceiverMode on the interpreter and TurboFan call.
>
> As a side effect, the "normal" Call bytecode now becomes a rare case
> (only with calls and super property calls), so we get rid of its 0-2
> argument special cases and modify CallProperty[N] to use the
> NotNullOrUndefined ConvertReceiverMode.
>
> Change-Id: I9374a32fefd66fc0251b5193bae7a6b7dc31eefc
> Reviewed-on: https://chromium-review.googlesource.com/463287
> Commit-Queue: Leszek Swirski <leszeks@chromium.org>
> Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#44530}
TBR=rmcilroy@chromium.org,mstarzinger@chromium.org,leszeks@chromium.org,v8-reviews@googlegroups.com,v8-mips-ports@googlegroups.com,v8-ppc-ports@googlegroups.com,v8-x87-ports@googlegroups.com,bmeurer@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: I7629dec609d0ec938ce7105d6c1c74884e5f9272
Reviewed-on: https://chromium-review.googlesource.com/474744
Commit-Queue: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Cr-Commit-Position: refs/heads/master@{#44548}
2017-04-11 06:26:10 +00:00
|
|
|
{{B(Wide), B(Call), R16(134), R16(135), U16(10), U16(177)},
|
2016-07-15 12:03:04 +00:00
|
|
|
10,
|
|
|
|
0,
|
Revert "[ignition] Add call bytecodes for undefined receiver"
This reverts commit 751e89359123df4f6e5f0b28b89fcdf63ed16a77.
Reason for revert: Breaks layout tests:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/14885
See:
https://github.com/v8/v8/wiki/Blink-layout-tests
Original change's description:
> [ignition] Add call bytecodes for undefined receiver
>
> Adds a collection of call bytecodes which have an implicit undefined
> receiver argument, for cases such as global calls where we know that the
> receiver has to be undefined. This way we can skip an LdaUndefined,
> decrease bytecode register pressure, and set a more accurate
> ConvertReceiverMode on the interpreter and TurboFan call.
>
> As a side effect, the "normal" Call bytecode now becomes a rare case
> (only with calls and super property calls), so we get rid of its 0-2
> argument special cases and modify CallProperty[N] to use the
> NotNullOrUndefined ConvertReceiverMode.
>
> Change-Id: I9374a32fefd66fc0251b5193bae7a6b7dc31eefc
> Reviewed-on: https://chromium-review.googlesource.com/463287
> Commit-Queue: Leszek Swirski <leszeks@chromium.org>
> Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#44530}
TBR=rmcilroy@chromium.org,mstarzinger@chromium.org,leszeks@chromium.org,v8-reviews@googlegroups.com,v8-mips-ports@googlegroups.com,v8-ppc-ports@googlegroups.com,v8-x87-ports@googlegroups.com,bmeurer@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: I7629dec609d0ec938ce7105d6c1c74884e5f9272
Reviewed-on: https://chromium-review.googlesource.com/474744
Commit-Queue: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Cr-Commit-Position: refs/heads/master@{#44548}
2017-04-11 06:26:10 +00:00
|
|
|
"Call.Wide r134, r135-r144, [177]"},
|
2016-10-05 16:14:12 +00:00
|
|
|
{{B(ForInPrepare), R8(10), R8(11)},
|
|
|
|
3,
|
|
|
|
0,
|
|
|
|
" ForInPrepare r10, r11-r13"},
|
2017-02-23 14:25:45 +00:00
|
|
|
{{B(CallRuntime), U16(Runtime::FunctionId::kIsDate), R8(0), U8(0)},
|
2016-10-05 16:14:12 +00:00
|
|
|
5,
|
|
|
|
0,
|
2017-02-23 14:25:45 +00:00
|
|
|
" CallRuntime [IsDate], r0-r0"},
|
2016-07-15 12:03:04 +00:00
|
|
|
{{B(Ldar),
|
|
|
|
static_cast<uint8_t>(Register::FromParameterIndex(2, 3).ToOperand())},
|
|
|
|
2,
|
|
|
|
3,
|
|
|
|
" Ldar a1"},
|
2016-08-09 10:30:03 +00:00
|
|
|
{{B(Wide), B(CreateObjectLiteral), U16(513), U16(1027), U8(165),
|
|
|
|
R16(137)},
|
|
|
|
9,
|
2016-07-15 12:03:04 +00:00
|
|
|
0,
|
2016-08-09 10:30:03 +00:00
|
|
|
"CreateObjectLiteral.Wide [513], [1027], #165, r137"},
|
2016-07-15 12:03:04 +00:00
|
|
|
{{B(ExtraWide), B(JumpIfNull), U32(123456789)},
|
|
|
|
6,
|
|
|
|
0,
|
|
|
|
"JumpIfNull.ExtraWide [123456789]"},
|
|
|
|
};
|
|
|
|
|
|
|
|
for (size_t i = 0; i < arraysize(cases); ++i) {
|
|
|
|
// Generate reference string by prepending formatted bytes.
|
|
|
|
std::stringstream expected_ss;
|
|
|
|
std::ios default_format(nullptr);
|
|
|
|
default_format.copyfmt(expected_ss);
|
|
|
|
// Match format of BytecodeDecoder::Decode() for byte representations.
|
|
|
|
expected_ss.fill('0');
|
|
|
|
expected_ss.flags(std::ios::right | std::ios::hex);
|
|
|
|
for (size_t b = 0; b < cases[i].length; b++) {
|
|
|
|
expected_ss << std::setw(2) << static_cast<uint32_t>(cases[i].bytecode[b])
|
|
|
|
<< ' ';
|
|
|
|
}
|
|
|
|
expected_ss.copyfmt(default_format);
|
|
|
|
expected_ss << cases[i].output;
|
|
|
|
|
|
|
|
// Generate decoded byte output.
|
|
|
|
std::stringstream actual_ss;
|
|
|
|
BytecodeDecoder::Decode(actual_ss, cases[i].bytecode,
|
|
|
|
cases[i].parameter_count);
|
|
|
|
|
|
|
|
// Compare.
|
|
|
|
CHECK_EQ(actual_ss.str(), expected_ss.str());
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
} // namespace interpreter
|
|
|
|
} // namespace internal
|
|
|
|
} // namespace v8
|