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

Unified Diff: runtime/vm/intermediate_language_arm.cc

Issue 626223002: Add unboxed Mint for X64 (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « runtime/vm/intermediate_language.h ('k') | runtime/vm/intermediate_language_ia32.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/intermediate_language_arm.cc
diff --git a/runtime/vm/intermediate_language_arm.cc b/runtime/vm/intermediate_language_arm.cc
index a6f1c76ef130c83a7e398244b3f38ae7bb3d0523..24d79d8bcb0c373eb07348e2d740806a4b55efbe 100644
--- a/runtime/vm/intermediate_language_arm.cc
+++ b/runtime/vm/intermediate_language_arm.cc
@@ -6285,15 +6285,6 @@ void ShiftMintOpInstr::EmitNativeCode(FlowGraphCompiler* compiler) {
ASSERT(locs()->in(1).constant().IsSmi());
const int32_t shift =
reinterpret_cast<int32_t>(locs()->in(1).constant().raw()) >> 1;
- if ((shift < 0) || (shift > kMintShiftCountLimit)) {
- __ b(deopt);
- return;
- } else if (shift == 0) {
- // Nothing to do for zero shift amount.
- __ mov(out_lo, Operand(left_lo));
- __ mov(out_hi, Operand(left_hi));
- return;
- }
switch (op_kind()) {
case Token::kSHR: {
if (shift < 32) {
@@ -6524,28 +6515,16 @@ void ShiftUint32OpInstr::EmitNativeCode(FlowGraphCompiler* compiler) {
ASSERT(constant.IsSmi());
const intptr_t shift_value = Smi::Cast(constant).Value();
- // Check constant shift value.
- if (shift_value == 0) {
- // Nothing to do.
- __ mov(out, Operand(left));
- } else if (shift_value < 0) {
- // Invalid shift value.
- __ b(deopt);
- } else if (shift_value > kShifterLimit) {
- // Result is 0.
- __ eor(out, out, Operand(out));
- } else {
- // Do the shift: (shift_value > 0) && (shift_value <= kShifterLimit).
- switch (op_kind()) {
- case Token::kSHR:
- __ Lsr(out, left, Operand(shift_value));
- break;
- case Token::kSHL:
- __ Lsl(out, left, Operand(shift_value));
- break;
- default:
- UNREACHABLE();
- }
+ // Do the shift: (shift_value > 0) && (shift_value <= kShifterLimit).
+ switch (op_kind()) {
+ case Token::kSHR:
+ __ Lsr(out, left, Operand(shift_value));
+ break;
+ case Token::kSHL:
+ __ Lsl(out, left, Operand(shift_value));
+ break;
+ default:
+ UNREACHABLE();
}
return;
}
« no previous file with comments | « runtime/vm/intermediate_language.h ('k') | runtime/vm/intermediate_language_ia32.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698