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

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

Issue 2477453005: MIPS[64]: Optimize kMips[64]Tst in code-generator-mips[64].cc (Closed)
Patch Set: Address code review remarks Created 4 years, 1 month 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/base/adapters.h" 5 #include "src/base/adapters.h"
6 #include "src/base/bits.h" 6 #include "src/base/bits.h"
7 #include "src/compiler/instruction-selector-impl.h" 7 #include "src/compiler/instruction-selector-impl.h"
8 #include "src/compiler/node-matchers.h" 8 #include "src/compiler/node-matchers.h"
9 #include "src/compiler/node-properties.h" 9 #include "src/compiler/node-properties.h"
10 10
(...skipping 25 matching lines...) Expand all
36 if (!m.HasValue()) return false; 36 if (!m.HasValue()) return false;
37 int32_t value = m.Value(); 37 int32_t value = m.Value();
38 switch (ArchOpcodeField::decode(opcode)) { 38 switch (ArchOpcodeField::decode(opcode)) {
39 case kMipsShl: 39 case kMipsShl:
40 case kMipsSar: 40 case kMipsSar:
41 case kMipsShr: 41 case kMipsShr:
42 return is_uint5(value); 42 return is_uint5(value);
43 case kMipsAdd: 43 case kMipsAdd:
44 case kMipsAnd: 44 case kMipsAnd:
45 case kMipsOr: 45 case kMipsOr:
46 case kMipsTst:
46 case kMipsSub: 47 case kMipsSub:
47 case kMipsXor: 48 case kMipsXor:
48 return is_uint16(value); 49 return is_uint16(value);
49 case kMipsLdc1: 50 case kMipsLdc1:
50 case kMipsSdc1: 51 case kMipsSdc1:
51 case kCheckedLoadFloat64: 52 case kCheckedLoadFloat64:
52 case kCheckedStoreFloat64: 53 case kCheckedStoreFloat64:
53 return std::numeric_limits<int16_t>::min() <= (value + kIntSize) && 54 return std::numeric_limits<int16_t>::min() <= (value + kIntSize) &&
54 std::numeric_limits<int16_t>::max() >= (value + kIntSize); 55 std::numeric_limits<int16_t>::max() >= (value + kIntSize);
55 default: 56 default:
(...skipping 1792 matching lines...) Expand 10 before | Expand all | Expand 10 after
1848 DCHECK(IsMipsArchVariant(kLoongson) || IsMipsArchVariant(kMips32r1) || 1849 DCHECK(IsMipsArchVariant(kLoongson) || IsMipsArchVariant(kMips32r1) ||
1849 IsMipsArchVariant(kMips32r2)); 1850 IsMipsArchVariant(kMips32r2));
1850 return MachineOperatorBuilder::AlignmentRequirements:: 1851 return MachineOperatorBuilder::AlignmentRequirements::
1851 NoUnalignedAccessSupport(); 1852 NoUnalignedAccessSupport();
1852 } 1853 }
1853 } 1854 }
1854 1855
1855 } // namespace compiler 1856 } // namespace compiler
1856 } // namespace internal 1857 } // namespace internal
1857 } // namespace v8 1858 } // namespace v8
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698