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

Side by Side Diff: src/compiler/register-allocator-verifier.cc

Issue 1287383003: Re-reland: Remove register index/code indirection (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Updated to ToT Created 5 years, 3 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 2014 the V8 project authors. All rights reserved. 1 // Copyright 2014 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/bit-vector.h" 5 #include "src/bit-vector.h"
6 #include "src/compiler/instruction.h" 6 #include "src/compiler/instruction.h"
7 #include "src/compiler/register-allocator-verifier.h" 7 #include "src/compiler/register-allocator-verifier.h"
8 8
9 namespace v8 { 9 namespace v8 {
10 namespace internal { 10 namespace internal {
(...skipping 200 matching lines...) Expand 10 before | Expand all | Expand 10 after
211 ? imm->inline_value() 211 ? imm->inline_value()
212 : imm->indexed_value(); 212 : imm->indexed_value();
213 CHECK_EQ(value, constraint->value_); 213 CHECK_EQ(value, constraint->value_);
214 return; 214 return;
215 } 215 }
216 case kRegister: 216 case kRegister:
217 CHECK(op->IsRegister()); 217 CHECK(op->IsRegister());
218 return; 218 return;
219 case kFixedRegister: 219 case kFixedRegister:
220 CHECK(op->IsRegister()); 220 CHECK(op->IsRegister());
221 CHECK_EQ(RegisterOperand::cast(op)->index(), constraint->value_); 221 CHECK_EQ(RegisterOperand::cast(op)->GetDoubleRegister().code(),
222 constraint->value_);
222 return; 223 return;
223 case kDoubleRegister: 224 case kDoubleRegister:
224 CHECK(op->IsDoubleRegister()); 225 CHECK(op->IsDoubleRegister());
225 return; 226 return;
226 case kFixedDoubleRegister: 227 case kFixedDoubleRegister:
227 CHECK(op->IsDoubleRegister()); 228 CHECK(op->IsDoubleRegister());
228 CHECK_EQ(DoubleRegisterOperand::cast(op)->index(), constraint->value_); 229 CHECK_EQ(DoubleRegisterOperand::cast(op)->GetDoubleRegister().code(),
230 constraint->value_);
229 return; 231 return;
230 case kFixedSlot: 232 case kFixedSlot:
231 CHECK(op->IsStackSlot()); 233 CHECK(op->IsStackSlot());
232 CHECK_EQ(StackSlotOperand::cast(op)->index(), constraint->value_); 234 CHECK_EQ(StackSlotOperand::cast(op)->index(), constraint->value_);
233 return; 235 return;
234 case kSlot: 236 case kSlot:
235 CHECK(op->IsStackSlot()); 237 CHECK(op->IsStackSlot());
236 return; 238 return;
237 case kDoubleSlot: 239 case kDoubleSlot:
238 CHECK(op->IsDoubleStackSlot()); 240 CHECK(op->IsDoubleStackSlot());
(...skipping 457 matching lines...) Expand 10 before | Expand all | Expand 10 after
696 int virtual_register = op_constraints[count].virtual_register_; 698 int virtual_register = op_constraints[count].virtual_register_;
697 current->Define(zone(), instr->OutputAt(i), virtual_register); 699 current->Define(zone(), instr->OutputAt(i), virtual_register);
698 } 700 }
699 } 701 }
700 } 702 }
701 } 703 }
702 704
703 } // namespace compiler 705 } // namespace compiler
704 } // namespace internal 706 } // namespace internal
705 } // namespace v8 707 } // namespace v8
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698