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

Side by Side Diff: src/arm64/macro-assembler-arm64.cc

Issue 299373005: Avoid HeapObject check in HStoreNamedField. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Fix condition on arm64. Created 6 years, 6 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 | Annotate | Revision Log
« no previous file with comments | « src/arm64/lithium-codegen-arm64.cc ('k') | src/hydrogen.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 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 "v8.h" 5 #include "v8.h"
6 6
7 #if V8_TARGET_ARCH_ARM64 7 #if V8_TARGET_ARCH_ARM64
8 8
9 #include "bootstrapper.h" 9 #include "bootstrapper.h"
10 #include "codegen.h" 10 #include "codegen.h"
(...skipping 541 matching lines...) Expand 10 before | Expand all | Expand 10 after
552 ASSERT(!r.IsDouble()); 552 ASSERT(!r.IsDouble());
553 553
554 if (r.IsInteger8() || r.IsUInteger8()) { 554 if (r.IsInteger8() || r.IsUInteger8()) {
555 Strb(rt, addr); 555 Strb(rt, addr);
556 } else if (r.IsInteger16() || r.IsUInteger16()) { 556 } else if (r.IsInteger16() || r.IsUInteger16()) {
557 Strh(rt, addr); 557 Strh(rt, addr);
558 } else if (r.IsInteger32()) { 558 } else if (r.IsInteger32()) {
559 Str(rt.W(), addr); 559 Str(rt.W(), addr);
560 } else { 560 } else {
561 ASSERT(rt.Is64Bits()); 561 ASSERT(rt.Is64Bits());
562 if (r.IsHeapObject()) {
563 AssertNotSmi(rt);
564 } else if (r.IsSmi()) {
565 AssertSmi(rt);
566 }
562 Str(rt, addr); 567 Str(rt, addr);
563 } 568 }
564 } 569 }
565 570
566 571
567 bool MacroAssembler::NeedExtraInstructionsOrRegisterBranch( 572 bool MacroAssembler::NeedExtraInstructionsOrRegisterBranch(
568 Label *label, ImmBranchType b_type) { 573 Label *label, ImmBranchType b_type) {
569 bool need_longer_range = false; 574 bool need_longer_range = false;
570 // There are two situations in which we care about the offset being out of 575 // There are two situations in which we care about the offset being out of
571 // range: 576 // range:
(...skipping 4683 matching lines...) Expand 10 before | Expand all | Expand 10 after
5255 } 5260 }
5256 } 5261 }
5257 5262
5258 5263
5259 #undef __ 5264 #undef __
5260 5265
5261 5266
5262 } } // namespace v8::internal 5267 } } // namespace v8::internal
5263 5268
5264 #endif // V8_TARGET_ARCH_ARM64 5269 #endif // V8_TARGET_ARCH_ARM64
OLDNEW
« no previous file with comments | « src/arm64/lithium-codegen-arm64.cc ('k') | src/hydrogen.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698