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

Side by Side Diff: src/compiler/arm64/instruction-selector-arm64.cc

Issue 1779713009: Implement optional turbofan UnalignedLoad and UnalignedStore operators (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Unaligned access simulate using load/shift/or and store/shift/and Created 4 years, 8 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/compiler/instruction-selector-impl.h" 5 #include "src/compiler/instruction-selector-impl.h"
6 #include "src/compiler/node-matchers.h" 6 #include "src/compiler/node-matchers.h"
7 #include "src/compiler/node-properties.h" 7 #include "src/compiler/node-properties.h"
8 8
9 namespace v8 { 9 namespace v8 {
10 namespace internal { 10 namespace internal {
(...skipping 467 matching lines...) Expand 10 before | Expand all | Expand 10 after
478 if (g.CanBeImmediate(index, immediate_mode)) { 478 if (g.CanBeImmediate(index, immediate_mode)) {
479 Emit(opcode | AddressingModeField::encode(kMode_MRI), g.NoOutput(), 479 Emit(opcode | AddressingModeField::encode(kMode_MRI), g.NoOutput(),
480 g.UseRegister(base), g.UseImmediate(index), g.UseRegister(value)); 480 g.UseRegister(base), g.UseImmediate(index), g.UseRegister(value));
481 } else { 481 } else {
482 Emit(opcode | AddressingModeField::encode(kMode_MRR), g.NoOutput(), 482 Emit(opcode | AddressingModeField::encode(kMode_MRR), g.NoOutput(),
483 g.UseRegister(base), g.UseRegister(index), g.UseRegister(value)); 483 g.UseRegister(base), g.UseRegister(index), g.UseRegister(value));
484 } 484 }
485 } 485 }
486 } 486 }
487 487
488 // Architecture supports unaligned access, therefore VisitLoad is used instead
489 void InstructionSelector::VisitUnalignedLoad(Node* node) { UNREACHABLE(); }
490
491 // Architecture supports unaligned access, therefore VisitStore is used instead
492 void InstructionSelector::VisitUnalignedStore(Node* node) { UNREACHABLE(); }
488 493
489 void InstructionSelector::VisitCheckedLoad(Node* node) { 494 void InstructionSelector::VisitCheckedLoad(Node* node) {
490 CheckedLoadRepresentation load_rep = CheckedLoadRepresentationOf(node->op()); 495 CheckedLoadRepresentation load_rep = CheckedLoadRepresentationOf(node->op());
491 Arm64OperandGenerator g(this); 496 Arm64OperandGenerator g(this);
492 Node* const buffer = node->InputAt(0); 497 Node* const buffer = node->InputAt(0);
493 Node* const offset = node->InputAt(1); 498 Node* const offset = node->InputAt(1);
494 Node* const length = node->InputAt(2); 499 Node* const length = node->InputAt(2);
495 ArchOpcode opcode = kArchNop; 500 ArchOpcode opcode = kArchNop;
496 switch (load_rep.representation()) { 501 switch (load_rep.representation()) {
497 case MachineRepresentation::kWord8: 502 case MachineRepresentation::kWord8:
(...skipping 1768 matching lines...) Expand 10 before | Expand all | Expand 10 after
2266 MachineOperatorBuilder::kWord32ShiftIsSafe | 2271 MachineOperatorBuilder::kWord32ShiftIsSafe |
2267 MachineOperatorBuilder::kInt32DivIsSafe | 2272 MachineOperatorBuilder::kInt32DivIsSafe |
2268 MachineOperatorBuilder::kUint32DivIsSafe | 2273 MachineOperatorBuilder::kUint32DivIsSafe |
2269 MachineOperatorBuilder::kWord32ReverseBits | 2274 MachineOperatorBuilder::kWord32ReverseBits |
2270 MachineOperatorBuilder::kWord64ReverseBits; 2275 MachineOperatorBuilder::kWord64ReverseBits;
2271 } 2276 }
2272 2277
2273 } // namespace compiler 2278 } // namespace compiler
2274 } // namespace internal 2279 } // namespace internal
2275 } // namespace v8 2280 } // namespace v8
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698