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

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: src/compiler/x64/instruction-scheduler-x64.cc

Issue 1375253002: [WIP][turbofan] Instruction scheduler for Turbofan. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Addressed issues and add support for a few other platforms. Created 5 years 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
« src/compiler/instruction-scheduler.h ('K') | « src/compiler/x64/code-generator-x64.cc ('k') | src/compiler/x87/code-generator-x87.cc » ('j') | src/globals.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright 2015 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "src/compiler/instruction-scheduler.h"
6
7 namespace v8 {
8 namespace internal {
9 namespace compiler {
10
11 int InstructionScheduler::GetTargetInstructionFlags(
12 const Instruction* instr) const {
13 switch (instr->arch_opcode()) {
14 case kX64Add:
15 case kX64Add32:
16 case kX64And:
17 case kX64And32:
18 case kX64Cmp:
19 case kX64Cmp32:
20 case kX64Test:
21 case kX64Test32:
22 case kX64Or:
23 case kX64Or32:
24 case kX64Xor:
25 case kX64Xor32:
26 case kX64Sub:
27 case kX64Sub32:
28 case kX64Imul:
29 case kX64Imul32:
30 case kX64ImulHigh32:
31 case kX64UmulHigh32:
32 case kX64Idiv:
33 case kX64Idiv32:
34 case kX64Udiv:
35 case kX64Udiv32:
36 case kX64Not:
37 case kX64Not32:
38 case kX64Neg:
39 case kX64Neg32:
40 case kX64Shl:
41 case kX64Shl32:
42 case kX64Shr:
43 case kX64Shr32:
44 case kX64Sar:
45 case kX64Sar32:
46 case kX64Ror:
47 case kX64Ror32:
48 case kX64Lzcnt:
49 case kX64Lzcnt32:
50 case kX64Tzcnt:
51 case kX64Tzcnt32:
52 case kX64Popcnt:
53 case kX64Popcnt32:
54 case kSSEFloat32Cmp:
55 case kSSEFloat32Add:
56 case kSSEFloat32Sub:
57 case kSSEFloat32Mul:
58 case kSSEFloat32Div:
59 case kSSEFloat32Abs:
60 case kSSEFloat32Neg:
61 case kSSEFloat32Sqrt:
62 case kSSEFloat32Max:
63 case kSSEFloat32Min:
64 case kSSEFloat32ToFloat64:
65 case kSSEFloat64Cmp:
66 case kSSEFloat64Add:
67 case kSSEFloat64Sub:
68 case kSSEFloat64Mul:
69 case kSSEFloat64Div:
70 case kSSEFloat64Mod:
71 case kSSEFloat64Abs:
72 case kSSEFloat64Neg:
73 case kSSEFloat64Sqrt:
74 case kSSEFloat64Round:
75 case kSSEFloat64Max:
76 case kSSEFloat64Min:
77 case kSSEFloat64ToFloat32:
78 case kSSEFloat64ToInt32:
79 case kSSEFloat64ToUint32:
80 case kSSEFloat64ToInt64:
81 case kSSEFloat64ToUint64:
82 case kSSEInt32ToFloat64:
83 case kSSEInt64ToFloat32:
84 case kSSEInt64ToFloat64:
85 case kSSEUint64ToFloat32:
86 case kSSEUint64ToFloat64:
87 case kSSEUint32ToFloat64:
88 case kSSEFloat64ExtractLowWord32:
89 case kSSEFloat64ExtractHighWord32:
90 case kSSEFloat64InsertLowWord32:
91 case kSSEFloat64InsertHighWord32:
92 case kSSEFloat64LoadLowWord32:
93 case kAVXFloat32Cmp:
94 case kAVXFloat32Add:
95 case kAVXFloat32Sub:
96 case kAVXFloat32Mul:
97 case kAVXFloat32Div:
98 case kAVXFloat32Max:
99 case kAVXFloat32Min:
100 case kAVXFloat64Cmp:
101 case kAVXFloat64Add:
102 case kAVXFloat64Sub:
103 case kAVXFloat64Mul:
104 case kAVXFloat64Div:
105 case kAVXFloat64Max:
106 case kAVXFloat64Min:
107 case kAVXFloat64Abs:
108 case kAVXFloat64Neg:
109 case kAVXFloat32Abs:
110 case kAVXFloat32Neg:
111 case kX64BitcastFI:
112 case kX64BitcastDL:
113 case kX64BitcastIF:
114 case kX64BitcastLD:
115 case kX64Lea32:
116 case kX64Lea:
117 case kX64Dec32:
118 case kX64Inc32:
119 // TODO(all): Do we need to check if these instructions read/store
120 // stack slots?
Jarin 2015/11/24 11:43:03 Yeah, these instructions can actually load/store a
baptiste.afsa1 2015/12/01 09:08:43 Done.
121 return kNoOpcodeFlags;
122
123 case kX64Movsxbl:
124 case kX64Movzxbl:
125 case kX64Movsxwl:
126 case kX64Movzxwl:
127 case kX64Movsxlq:
128 DCHECK(instr->InputCount() >= 1);
129 return instr->InputAt(0)->IsRegister() ? kNoOpcodeFlags
130 : kIsLoadOperation;
131
132 case kX64Movb:
133 case kX64Movw:
134 return kHasSideEffect;
135
136 case kX64Movl:
137 if (instr->HasOutput()) {
138 DCHECK(instr->InputCount() >= 1);
139 return instr->InputAt(0)->IsRegister() ? kNoOpcodeFlags
140 : kIsLoadOperation;
141 } else {
142 return kHasSideEffect;
143 }
144
145 case kX64Movq:
146 case kX64Movsd:
147 case kX64Movss:
148 return instr->HasOutput() ? kIsLoadOperation : kHasSideEffect;
149
150 case kX64StackCheck:
151 return kIsLoadOperation;
152
153 case kX64Push:
154 case kX64Poke:
155 return kHasSideEffect;
156
157 #define CASE(Name) case k##Name:
158 COMMON_ARCH_OPCODE_LIST(CASE)
159 #undef CASE
160 // Already covered in architecture independent code.
161 UNREACHABLE();
162 }
163 }
164
165
166 int InstructionScheduler::GetInstructionLatency(const Instruction* instr) {
167 // TODO(all): Add instruction cost modeling.
168 return 1;
169 }
170
171 } // namespace compiler
172 } // namespace internal
173 } // namespace v8
OLDNEW
« src/compiler/instruction-scheduler.h ('K') | « src/compiler/x64/code-generator-x64.cc ('k') | src/compiler/x87/code-generator-x87.cc » ('j') | src/globals.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698