Thanks for cleaning this up! The 'unimplemented instruction' confused me already a couple of times.
Den 2. dec. 2011 15.12 skrev <[email protected]>: > Reviewers: Michael Starzinger, > > Description: > Clean up multi byte nop support on x64 to more closely match IA32. > Fix missing instruction in disassembler. > Fix wrong disassembly of multi-byte NOP on x64 > Add test of disassembler on 64 bit! > > Please review this at > http://codereview.chromium.**org/8773039/<http://codereview.chromium.org/8773039/> > > SVN Base: > http://v8.googlecode.com/svn/**branches/bleeding_edge/<http://v8.googlecode.com/svn/branches/bleeding_edge/> > > Affected files: > M src/x64/assembler-x64.h > M src/x64/assembler-x64.cc > M src/x64/debug-x64.cc > M src/x64/deoptimizer-x64.cc > M src/x64/disasm-x64.cc > M src/x64/lithium-codegen-x64.cc > M test/cctest/SConscript > M test/cctest/test-assembler-**x64.cc > A test/cctest/test-disasm-x64.cc > > > -- > v8-dev mailing list > [email protected] > http://groups.google.com/**group/v8-dev<http://groups.google.com/group/v8-dev> > -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
