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

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

Issue 2591643005: S390 [Crankshaft]: Use LoadPositive for MathAbs (Closed)
Patch Set: Fix for John's comment Created 3 years, 12 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 <assert.h> // For assert 5 #include <assert.h> // For assert
6 #include <limits.h> // For LONG_MIN, LONG_MAX. 6 #include <limits.h> // For LONG_MIN, LONG_MAX.
7 7
8 #if V8_TARGET_ARCH_S390 8 #if V8_TARGET_ARCH_S390
9 9
10 #include "src/base/bits.h" 10 #include "src/base/bits.h"
(...skipping 4114 matching lines...) Expand 10 before | Expand all | Expand 10 after
4125 lgf(dst, opnd); // 64<-32 4125 lgf(dst, opnd); // 64<-32
4126 #else 4126 #else
4127 if (is_uint12(opnd.offset())) { 4127 if (is_uint12(opnd.offset())) {
4128 l(dst, opnd); 4128 l(dst, opnd);
4129 } else { 4129 } else {
4130 ly(dst, opnd); 4130 ly(dst, opnd);
4131 } 4131 }
4132 #endif 4132 #endif
4133 } 4133 }
4134 4134
4135 void MacroAssembler::LoadPositiveP(Register result, Register input) {
4136 #if V8_TARGET_ARCH_S390X
4137 lpgr(result, input);
4138 #else
4139 lpr(result, input);
4140 #endif
4141 }
4142
4143 void MacroAssembler::LoadPositive32(Register result, Register input) {
4144 lpr(result, input);
4145 lgfr(result, result);
4146 }
4147
4135 //----------------------------------------------------------------------------- 4148 //-----------------------------------------------------------------------------
4136 // Compare Helpers 4149 // Compare Helpers
4137 //----------------------------------------------------------------------------- 4150 //-----------------------------------------------------------------------------
4138 4151
4139 // Compare 32-bit Register vs Register 4152 // Compare 32-bit Register vs Register
4140 void MacroAssembler::Cmp32(Register src1, Register src2) { cr_z(src1, src2); } 4153 void MacroAssembler::Cmp32(Register src1, Register src2) { cr_z(src1, src2); }
4141 4154
4142 // Compare Pointer Sized Register vs Register 4155 // Compare Pointer Sized Register vs Register
4143 void MacroAssembler::CmpP(Register src1, Register src2) { 4156 void MacroAssembler::CmpP(Register src1, Register src2) {
4144 #if V8_TARGET_ARCH_S390X 4157 #if V8_TARGET_ARCH_S390X
(...skipping 935 matching lines...) Expand 10 before | Expand all | Expand 10 after
5080 } 5093 }
5081 if (mag.shift > 0) ShiftRightArith(result, result, Operand(mag.shift)); 5094 if (mag.shift > 0) ShiftRightArith(result, result, Operand(mag.shift));
5082 ExtractBit(r0, dividend, 31); 5095 ExtractBit(r0, dividend, 31);
5083 AddP(result, r0); 5096 AddP(result, r0);
5084 } 5097 }
5085 5098
5086 } // namespace internal 5099 } // namespace internal
5087 } // namespace v8 5100 } // namespace v8
5088 5101
5089 #endif // V8_TARGET_ARCH_S390 5102 #endif // V8_TARGET_ARCH_S390
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698