AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

[ia32] Fix disassembler tests to test instruction disassembly

Browse Source 
Change-Id: I1fe7ba0a942c6da2d1c85c2391762cb434098cf1
Reviewed-on: https://chromium-review.googlesource.com/907346
Reviewed-by: Jakob Kummerow <jkummerow@chromium.org>
Commit-Queue: Deepti Gandluri <gdeepti@chromium.org>
Cr-Commit-Position: refs/heads/master@{#51165}
This commit is contained in:
Deepti Gandluri 2018-02-07 15:50:26 -08:00 committed by Commit Bot
parent 601f5f7897
commit 2b67f8de57
2 changed files with 25 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/ia32/disasm-ia32.cc
View File

@ -2564,6 +2564,11 @@ int Disassembler::InstructionDecode(v8::internal::Vector<char> buffer,
return d.InstructionDecode(buffer, instruction); return d.InstructionDecode(buffer, instruction);
} }
int Disassembler::InstructionDecodeForTesting(v8::internal::Vector<char> buffer,
byte* instruction) {
DisassemblerIA32 d(converter_, true /*crash if unimplemented*/);
return d.InstructionDecode(buffer, instruction);
}
// The IA-32 assembler does not currently use constant pools. // The IA-32 assembler does not currently use constant pools.
int Disassembler::ConstantPoolSizeAt(byte* instruction) { return -1; } int Disassembler::ConstantPoolSizeAt(byte* instruction) { return -1; }

21
test/cctest/test-disasm-ia32.cc
View File

@ -42,7 +42,26 @@ namespace internal {
#define __ assm. #define __ assm.
void Disassemble(FILE* f, byte* begin, byte* end) {
disasm::NameConverter converter;
disasm::Disassembler d(converter);
for (byte* pc = begin; pc < end;) {
v8::internal::EmbeddedVector<char, 128> buffer;
buffer[0] = '\0';
byte* prev_pc = pc;
pc += d.InstructionDecodeForTesting(buffer, pc);
fprintf(f, "%p", static_cast<void*>(prev_pc));
fprintf(f, " ");
for (byte* bp = prev_pc; bp < pc; bp++) {
fprintf(f, "%02x", *bp);
}
for (int i = 6 - (pc - prev_pc); i >= 0; i--) {
fprintf(f, " ");
}
fprintf(f, " %s\n", buffer.start());
}
}
static void DummyStaticFunction(Object* result) { static void DummyStaticFunction(Object* result) {
} }
@ -865,7 +884,7 @@ TEST(DisasmIa320) {
code->Print(os); code->Print(os);
byte* begin = code->instruction_start(); byte* begin = code->instruction_start();
byte* end = begin + code->instruction_size(); byte* end = begin + code->instruction_size();
disasm::Disassembler::Disassemble(stdout, begin, end); Disassemble(stdout, begin, end);
#endif #endif
} }