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

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

Issue 2623633003: [Atomics] Make Atomics.exchange a builtin using TF (Closed)
Patch Set: [Atomics] Make Atomics.exchange a builtin using TF Created 3 years, 11 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
OLDNEW
1 // Copyright 2015 the V8 project authors. All rights reserved. 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 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 "src/compiler/instruction-scheduler.h" 5 #include "src/compiler/instruction-scheduler.h"
6 6
7 namespace v8 { 7 namespace v8 {
8 namespace internal { 8 namespace internal {
9 namespace compiler { 9 namespace compiler {
10 10
(...skipping 160 matching lines...) Expand 10 before | Expand all | Expand 10 after
171 case kX64Movss: 171 case kX64Movss:
172 return instr->HasOutput() ? kIsLoadOperation : kHasSideEffect; 172 return instr->HasOutput() ? kIsLoadOperation : kHasSideEffect;
173 173
174 case kX64StackCheck: 174 case kX64StackCheck:
175 return kIsLoadOperation; 175 return kIsLoadOperation;
176 176
177 case kX64Push: 177 case kX64Push:
178 case kX64Poke: 178 case kX64Poke:
179 return kHasSideEffect; 179 return kHasSideEffect;
180 180
181 case kX64Xchgb:
182 case kX64Xchgw:
183 case kX64Xchgl:
184 return kIsLoadOperation | kHasSideEffect;
binji 2017/01/17 21:45:50 Where did this move to?
aseemgarg 2017/01/21 08:40:37 These are removed. Instead we are now relying on t
185
186 #define CASE(Name) case k##Name: 181 #define CASE(Name) case k##Name:
187 COMMON_ARCH_OPCODE_LIST(CASE) 182 COMMON_ARCH_OPCODE_LIST(CASE)
188 #undef CASE 183 #undef CASE
189 // Already covered in architecture independent code. 184 // Already covered in architecture independent code.
190 UNREACHABLE(); 185 UNREACHABLE();
191 } 186 }
192 187
193 UNREACHABLE(); 188 UNREACHABLE();
194 return kNoOpcodeFlags; 189 return kNoOpcodeFlags;
195 } 190 }
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
265 case kArchTruncateDoubleToI: 260 case kArchTruncateDoubleToI:
266 return 6; 261 return 6;
267 default: 262 default:
268 return 1; 263 return 1;
269 } 264 }
270 } 265 }
271 266
272 } // namespace compiler 267 } // namespace compiler
273 } // namespace internal 268 } // namespace internal
274 } // namespace v8 269 } // namespace v8
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698