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

Side by Side Diff: runtime/vm/intermediate_language_arm64.cc

Issue 1961393002: VM: Optimized code for all of [External]{One|Two}ByteString::codeUnitAt. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: addressed comment Created 4 years, 7 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 | « runtime/vm/intermediate_language_arm.cc ('k') | runtime/vm/intermediate_language_ia32.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 (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #include "vm/globals.h" // Needed here to get TARGET_ARCH_ARM64. 5 #include "vm/globals.h" // Needed here to get TARGET_ARCH_ARM64.
6 #if defined(TARGET_ARCH_ARM64) 6 #if defined(TARGET_ARCH_ARM64)
7 7
8 #include "vm/intermediate_language.h" 8 #include "vm/intermediate_language.h"
9 9
10 #include "vm/compiler.h" 10 #include "vm/compiler.h"
(...skipping 804 matching lines...) Expand 10 before | Expand all | Expand 10 after
815 ExternalLabel label(entry); 815 ExternalLabel label(entry);
816 __ LoadNativeEntry(R5, &label); 816 __ LoadNativeEntry(R5, &label);
817 compiler->GenerateCall(token_pos(), 817 compiler->GenerateCall(token_pos(),
818 *stub_entry, 818 *stub_entry,
819 RawPcDescriptors::kOther, 819 RawPcDescriptors::kOther,
820 locs()); 820 locs());
821 __ Pop(result); 821 __ Pop(result);
822 } 822 }
823 823
824 824
825 LocationSummary* StringFromCharCodeInstr::MakeLocationSummary(Zone* zone, 825 LocationSummary* OneByteStringFromCharCodeInstr::MakeLocationSummary(
826 bool opt) const { 826 Zone* zone, bool opt) const {
827 const intptr_t kNumInputs = 1; 827 const intptr_t kNumInputs = 1;
828 // TODO(fschneider): Allow immediate operands for the char code. 828 // TODO(fschneider): Allow immediate operands for the char code.
829 return LocationSummary::Make(zone, 829 return LocationSummary::Make(zone,
830 kNumInputs, 830 kNumInputs,
831 Location::RequiresRegister(), 831 Location::RequiresRegister(),
832 LocationSummary::kNoCall); 832 LocationSummary::kNoCall);
833 } 833 }
834 834
835 835
836 void StringFromCharCodeInstr::EmitNativeCode(FlowGraphCompiler* compiler) { 836 void OneByteStringFromCharCodeInstr::EmitNativeCode(
837 FlowGraphCompiler* compiler) {
837 ASSERT(compiler->is_optimizing()); 838 ASSERT(compiler->is_optimizing());
838 const Register char_code = locs()->in(0).reg(); 839 const Register char_code = locs()->in(0).reg();
839 const Register result = locs()->out(0).reg(); 840 const Register result = locs()->out(0).reg();
840 841
841 __ ldr(result, Address(THR, Thread::predefined_symbols_address_offset())); 842 __ ldr(result, Address(THR, Thread::predefined_symbols_address_offset()));
842 __ AddImmediate( 843 __ AddImmediate(
843 result, result, Symbols::kNullCharCodeSymbolOffset * kWordSize); 844 result, result, Symbols::kNullCharCodeSymbolOffset * kWordSize);
844 __ SmiUntag(TMP, char_code); // Untag to use scaled adress mode. 845 __ SmiUntag(TMP, char_code); // Untag to use scaled adress mode.
845 __ ldr(result, Address(result, TMP, UXTX, Address::Scaled)); 846 __ ldr(result, Address(result, TMP, UXTX, Address::Scaled));
846 } 847 }
(...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
961 962
962 case kTypedDataInt8ArrayCid: 963 case kTypedDataInt8ArrayCid:
963 case kTypedDataUint8ArrayCid: 964 case kTypedDataUint8ArrayCid:
964 case kTypedDataUint8ClampedArrayCid: 965 case kTypedDataUint8ClampedArrayCid:
965 case kExternalTypedDataUint8ArrayCid: 966 case kExternalTypedDataUint8ArrayCid:
966 case kExternalTypedDataUint8ClampedArrayCid: 967 case kExternalTypedDataUint8ClampedArrayCid:
967 case kTypedDataInt16ArrayCid: 968 case kTypedDataInt16ArrayCid:
968 case kTypedDataUint16ArrayCid: 969 case kTypedDataUint16ArrayCid:
969 case kOneByteStringCid: 970 case kOneByteStringCid:
970 case kTwoByteStringCid: 971 case kTwoByteStringCid:
972 case kExternalOneByteStringCid:
973 case kExternalTwoByteStringCid:
971 case kTypedDataInt32ArrayCid: 974 case kTypedDataInt32ArrayCid:
972 case kTypedDataUint32ArrayCid: 975 case kTypedDataUint32ArrayCid:
973 return CompileType::FromCid(kSmiCid); 976 return CompileType::FromCid(kSmiCid);
974 977
975 default: 978 default:
976 UNIMPLEMENTED(); 979 UNIMPLEMENTED();
977 return CompileType::Dynamic(); 980 return CompileType::Dynamic();
978 } 981 }
979 } 982 }
980 983
981 984
982 Representation LoadIndexedInstr::representation() const { 985 Representation LoadIndexedInstr::representation() const {
983 switch (class_id_) { 986 switch (class_id_) {
984 case kArrayCid: 987 case kArrayCid:
985 case kImmutableArrayCid: 988 case kImmutableArrayCid:
986 case kTypedDataInt8ArrayCid: 989 case kTypedDataInt8ArrayCid:
987 case kTypedDataUint8ArrayCid: 990 case kTypedDataUint8ArrayCid:
988 case kTypedDataUint8ClampedArrayCid: 991 case kTypedDataUint8ClampedArrayCid:
989 case kExternalTypedDataUint8ArrayCid: 992 case kExternalTypedDataUint8ArrayCid:
990 case kExternalTypedDataUint8ClampedArrayCid: 993 case kExternalTypedDataUint8ClampedArrayCid:
991 case kTypedDataInt16ArrayCid: 994 case kTypedDataInt16ArrayCid:
992 case kTypedDataUint16ArrayCid: 995 case kTypedDataUint16ArrayCid:
993 case kOneByteStringCid: 996 case kOneByteStringCid:
994 case kTwoByteStringCid: 997 case kTwoByteStringCid:
998 case kExternalOneByteStringCid:
999 case kExternalTwoByteStringCid:
995 return kTagged; 1000 return kTagged;
996 case kTypedDataInt32ArrayCid: 1001 case kTypedDataInt32ArrayCid:
997 return kUnboxedInt32; 1002 return kUnboxedInt32;
998 case kTypedDataUint32ArrayCid: 1003 case kTypedDataUint32ArrayCid:
999 return kUnboxedUint32; 1004 return kUnboxedUint32;
1000 case kTypedDataFloat32ArrayCid: 1005 case kTypedDataFloat32ArrayCid:
1001 case kTypedDataFloat64ArrayCid: 1006 case kTypedDataFloat64ArrayCid:
1002 return kUnboxedDouble; 1007 return kUnboxedDouble;
1003 case kTypedDataInt32x4ArrayCid: 1008 case kTypedDataInt32x4ArrayCid:
1004 return kUnboxedInt32x4; 1009 return kUnboxedInt32x4;
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
1118 case kTypedDataInt8ArrayCid: 1123 case kTypedDataInt8ArrayCid:
1119 ASSERT(index_scale() == 1); 1124 ASSERT(index_scale() == 1);
1120 __ ldr(result, element_address, kByte); 1125 __ ldr(result, element_address, kByte);
1121 __ SmiTag(result); 1126 __ SmiTag(result);
1122 break; 1127 break;
1123 case kTypedDataUint8ArrayCid: 1128 case kTypedDataUint8ArrayCid:
1124 case kTypedDataUint8ClampedArrayCid: 1129 case kTypedDataUint8ClampedArrayCid:
1125 case kExternalTypedDataUint8ArrayCid: 1130 case kExternalTypedDataUint8ArrayCid:
1126 case kExternalTypedDataUint8ClampedArrayCid: 1131 case kExternalTypedDataUint8ClampedArrayCid:
1127 case kOneByteStringCid: 1132 case kOneByteStringCid:
1133 case kExternalOneByteStringCid:
1128 ASSERT(index_scale() == 1); 1134 ASSERT(index_scale() == 1);
1129 __ ldr(result, element_address, kUnsignedByte); 1135 __ ldr(result, element_address, kUnsignedByte);
1130 __ SmiTag(result); 1136 __ SmiTag(result);
1131 break; 1137 break;
1132 case kTypedDataInt16ArrayCid: 1138 case kTypedDataInt16ArrayCid:
1133 __ ldr(result, element_address, kHalfword); 1139 __ ldr(result, element_address, kHalfword);
1134 __ SmiTag(result); 1140 __ SmiTag(result);
1135 break; 1141 break;
1136 case kTypedDataUint16ArrayCid: 1142 case kTypedDataUint16ArrayCid:
1137 case kTwoByteStringCid: 1143 case kTwoByteStringCid:
1144 case kExternalTwoByteStringCid:
1138 __ ldr(result, element_address, kUnsignedHalfword); 1145 __ ldr(result, element_address, kUnsignedHalfword);
1139 __ SmiTag(result); 1146 __ SmiTag(result);
1140 break; 1147 break;
1141 default: 1148 default:
1142 ASSERT((class_id() == kArrayCid) || (class_id() == kImmutableArrayCid)); 1149 ASSERT((class_id() == kArrayCid) || (class_id() == kImmutableArrayCid));
1143 __ ldr(result, element_address); 1150 __ ldr(result, element_address);
1144 break; 1151 break;
1145 } 1152 }
1146 } 1153 }
1147 1154
(...skipping 4557 matching lines...) Expand 10 before | Expand all | Expand 10 after
5705 1, 5712 1,
5706 locs()); 5713 locs());
5707 __ Drop(1); 5714 __ Drop(1);
5708 __ Pop(result); 5715 __ Pop(result);
5709 } 5716 }
5710 5717
5711 5718
5712 } // namespace dart 5719 } // namespace dart
5713 5720
5714 #endif // defined TARGET_ARCH_ARM64 5721 #endif // defined TARGET_ARCH_ARM64
OLDNEW
« no previous file with comments | « runtime/vm/intermediate_language_arm.cc ('k') | runtime/vm/intermediate_language_ia32.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698