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

Side by Side Diff: src/crankshaft/s390/lithium-codegen-s390.cc

Issue 2651673009: s390: Fix DoArgumentsElements where should use CmpP (Closed)
Patch Set: 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 the V8 project authors. All rights reserved. 1 // Copyright 2014 the V8 project authors. All rights reserved.
2 // 2 //
3 // Use of this source code is governed by a BSD-style license that can be 3 // Use of this source code is governed by a BSD-style license that can be
4 // found in the LICENSE file. 4 // found in the LICENSE file.
5 5
6 #include "src/crankshaft/s390/lithium-codegen-s390.h" 6 #include "src/crankshaft/s390/lithium-codegen-s390.h"
7 7
8 #include "src/base/bits.h" 8 #include "src/base/bits.h"
9 #include "src/builtins/builtins-constructor.h" 9 #include "src/builtins/builtins-constructor.h"
10 #include "src/code-factory.h" 10 #include "src/code-factory.h"
(...skipping 3036 matching lines...) Expand 10 before | Expand all | Expand 10 after
3047 3047
3048 if (instr->hydrogen()->from_inlined()) { 3048 if (instr->hydrogen()->from_inlined()) {
3049 __ lay(result, MemOperand(sp, -2 * kPointerSize)); 3049 __ lay(result, MemOperand(sp, -2 * kPointerSize));
3050 } else if (instr->hydrogen()->arguments_adaptor()) { 3050 } else if (instr->hydrogen()->arguments_adaptor()) {
3051 // Check if the calling frame is an arguments adaptor frame. 3051 // Check if the calling frame is an arguments adaptor frame.
3052 Label done, adapted; 3052 Label done, adapted;
3053 __ LoadP(scratch, MemOperand(fp, StandardFrameConstants::kCallerFPOffset)); 3053 __ LoadP(scratch, MemOperand(fp, StandardFrameConstants::kCallerFPOffset));
3054 __ LoadP( 3054 __ LoadP(
3055 result, 3055 result,
3056 MemOperand(scratch, CommonFrameConstants::kContextOrFrameTypeOffset)); 3056 MemOperand(scratch, CommonFrameConstants::kContextOrFrameTypeOffset));
3057 __ CmpSmiLiteral(result, Smi::FromInt(StackFrame::ARGUMENTS_ADAPTOR), r0); 3057 __ LoadSmiLiteral(r0, Smi::FromInt(StackFrame::ARGUMENTS_ADAPTOR));
3058 __ CmpP(result, r0);
3058 3059
3059 // Result is the frame pointer for the frame if not adapted and for the real 3060 // Result is the frame pointer for the frame if not adapted and for the real
3060 // frame below the adaptor frame if adapted. 3061 // frame below the adaptor frame if adapted.
3061 __ beq(&adapted, Label::kNear); 3062 __ beq(&adapted, Label::kNear);
3062 __ LoadRR(result, fp); 3063 __ LoadRR(result, fp);
3063 __ b(&done, Label::kNear); 3064 __ b(&done, Label::kNear);
3064 3065
3065 __ bind(&adapted); 3066 __ bind(&adapted);
3066 __ LoadRR(result, scratch); 3067 __ LoadRR(result, scratch);
3067 __ bind(&done); 3068 __ bind(&done);
(...skipping 2485 matching lines...) Expand 10 before | Expand all | Expand 10 after
5553 __ LoadP(result, 5554 __ LoadP(result,
5554 FieldMemOperand(scratch, FixedArray::kHeaderSize - kPointerSize)); 5555 FieldMemOperand(scratch, FixedArray::kHeaderSize - kPointerSize));
5555 __ bind(deferred->exit()); 5556 __ bind(deferred->exit());
5556 __ bind(&done); 5557 __ bind(&done);
5557 } 5558 }
5558 5559
5559 #undef __ 5560 #undef __
5560 5561
5561 } // namespace internal 5562 } // namespace internal
5562 } // namespace v8 5563 } // namespace v8
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698