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

Side by Side Diff: src/x87/disasm-x87.cc

Issue 1724473004: X87: Emit memory operands for cmp and test on ia32 and x64 when it makes sense. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 10 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 unified diff | Download patch
« no previous file with comments | « src/x87/assembler-x87.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 the V8 project authors. All rights reserved. 1 // Copyright 2011 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <assert.h> 5 #include <assert.h>
6 #include <stdarg.h> 6 #include <stdarg.h>
7 #include <stdio.h> 7 #include <stdio.h>
8 8
9 #if V8_TARGET_ARCH_X87 9 #if V8_TARGET_ARCH_X87
10 10
(...skipping 10 matching lines...) Expand all
21 21
22 //------------------------------------------------------------------ 22 //------------------------------------------------------------------
23 // Tables 23 // Tables
24 //------------------------------------------------------------------ 24 //------------------------------------------------------------------
25 struct ByteMnemonic { 25 struct ByteMnemonic {
26 int b; // -1 terminates, otherwise must be in range (0..255) 26 int b; // -1 terminates, otherwise must be in range (0..255)
27 const char* mnem; 27 const char* mnem;
28 OperandOrder op_order_; 28 OperandOrder op_order_;
29 }; 29 };
30 30
31
32 static const ByteMnemonic two_operands_instr[] = { 31 static const ByteMnemonic two_operands_instr[] = {
33 {0x01, "add", OPER_REG_OP_ORDER}, 32 {0x01, "add", OPER_REG_OP_ORDER},
34 {0x03, "add", REG_OPER_OP_ORDER}, 33 {0x03, "add", REG_OPER_OP_ORDER},
35 {0x09, "or", OPER_REG_OP_ORDER}, 34 {0x09, "or", OPER_REG_OP_ORDER},
36 {0x0B, "or", REG_OPER_OP_ORDER}, 35 {0x0B, "or", REG_OPER_OP_ORDER},
37 {0x1B, "sbb", REG_OPER_OP_ORDER}, 36 {0x1B, "sbb", REG_OPER_OP_ORDER},
38 {0x21, "and", OPER_REG_OP_ORDER}, 37 {0x21, "and", OPER_REG_OP_ORDER},
39 {0x23, "and", REG_OPER_OP_ORDER}, 38 {0x23, "and", REG_OPER_OP_ORDER},
40 {0x29, "sub", OPER_REG_OP_ORDER}, 39 {0x29, "sub", OPER_REG_OP_ORDER},
41 {0x2A, "subb", REG_OPER_OP_ORDER}, 40 {0x2A, "subb", REG_OPER_OP_ORDER},
42 {0x2B, "sub", REG_OPER_OP_ORDER}, 41 {0x2B, "sub", REG_OPER_OP_ORDER},
43 {0x31, "xor", OPER_REG_OP_ORDER}, 42 {0x31, "xor", OPER_REG_OP_ORDER},
44 {0x33, "xor", REG_OPER_OP_ORDER}, 43 {0x33, "xor", REG_OPER_OP_ORDER},
45 {0x38, "cmpb", OPER_REG_OP_ORDER}, 44 {0x38, "cmpb", OPER_REG_OP_ORDER},
46 {0x3A, "cmpb", REG_OPER_OP_ORDER}, 45 {0x39, "cmp", OPER_REG_OP_ORDER},
47 {0x3B, "cmp", REG_OPER_OP_ORDER}, 46 {0x3A, "cmpb", REG_OPER_OP_ORDER},
48 {0x84, "test_b", REG_OPER_OP_ORDER}, 47 {0x3B, "cmp", REG_OPER_OP_ORDER},
49 {0x85, "test", REG_OPER_OP_ORDER}, 48 {0x84, "test_b", REG_OPER_OP_ORDER},
50 {0x87, "xchg", REG_OPER_OP_ORDER}, 49 {0x85, "test", REG_OPER_OP_ORDER},
51 {0x8A, "mov_b", REG_OPER_OP_ORDER}, 50 {0x87, "xchg", REG_OPER_OP_ORDER},
52 {0x8B, "mov", REG_OPER_OP_ORDER}, 51 {0x8A, "mov_b", REG_OPER_OP_ORDER},
53 {0x8D, "lea", REG_OPER_OP_ORDER}, 52 {0x8B, "mov", REG_OPER_OP_ORDER},
54 {-1, "", UNSET_OP_ORDER} 53 {0x8D, "lea", REG_OPER_OP_ORDER},
55 }; 54 {-1, "", UNSET_OP_ORDER}};
56
57 55
58 static const ByteMnemonic zero_operands_instr[] = { 56 static const ByteMnemonic zero_operands_instr[] = {
59 {0xC3, "ret", UNSET_OP_ORDER}, 57 {0xC3, "ret", UNSET_OP_ORDER},
60 {0xC9, "leave", UNSET_OP_ORDER}, 58 {0xC9, "leave", UNSET_OP_ORDER},
61 {0x90, "nop", UNSET_OP_ORDER}, 59 {0x90, "nop", UNSET_OP_ORDER},
62 {0xF4, "hlt", UNSET_OP_ORDER}, 60 {0xF4, "hlt", UNSET_OP_ORDER},
63 {0xCC, "int3", UNSET_OP_ORDER}, 61 {0xCC, "int3", UNSET_OP_ORDER},
64 {0x60, "pushad", UNSET_OP_ORDER}, 62 {0x60, "pushad", UNSET_OP_ORDER},
65 {0x61, "popad", UNSET_OP_ORDER}, 63 {0x61, "popad", UNSET_OP_ORDER},
66 {0x9C, "pushfd", UNSET_OP_ORDER}, 64 {0x9C, "pushfd", UNSET_OP_ORDER},
(...skipping 1730 matching lines...) Expand 10 before | Expand all | Expand 10 after
1797 fprintf(f, " "); 1795 fprintf(f, " ");
1798 } 1796 }
1799 fprintf(f, " %s\n", buffer.start()); 1797 fprintf(f, " %s\n", buffer.start());
1800 } 1798 }
1801 } 1799 }
1802 1800
1803 1801
1804 } // namespace disasm 1802 } // namespace disasm
1805 1803
1806 #endif // V8_TARGET_ARCH_X87 1804 #endif // V8_TARGET_ARCH_X87
OLDNEW
« no previous file with comments | « src/x87/assembler-x87.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698