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

Side by Side Diff: src/compiler/js-create-lowering.cc

Issue 2609173005: [V8] Run clang-tidy performance-for-range-copy check on src/ (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 | « src/compilation-statistics.cc ('k') | src/compiler/js-native-context-specialization.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 2016 the V8 project authors. All rights reserved. 1 // Copyright 2016 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/compiler/js-create-lowering.h" 5 #include "src/compiler/js-create-lowering.h"
6 6
7 #include "src/allocation-site-scopes.h" 7 #include "src/allocation-site-scopes.h"
8 #include "src/code-factory.h" 8 #include "src/code-factory.h"
9 #include "src/compilation-dependencies.h" 9 #include "src/compilation-dependencies.h"
10 #include "src/compiler/access-builder.h" 10 #include "src/compiler/access-builder.h"
(...skipping 1232 matching lines...) Expand 10 before | Expand all | Expand 10 after
1243 Type::OtherObject()); 1243 Type::OtherObject());
1244 builder.Store(AccessBuilder::ForMap(), boilerplate_map); 1244 builder.Store(AccessBuilder::ForMap(), boilerplate_map);
1245 builder.Store(AccessBuilder::ForJSObjectProperties(), properties); 1245 builder.Store(AccessBuilder::ForJSObjectProperties(), properties);
1246 builder.Store(AccessBuilder::ForJSObjectElements(), elements); 1246 builder.Store(AccessBuilder::ForJSObjectElements(), elements);
1247 if (boilerplate_map->IsJSArrayMap()) { 1247 if (boilerplate_map->IsJSArrayMap()) {
1248 Handle<JSArray> boilerplate_array = Handle<JSArray>::cast(boilerplate); 1248 Handle<JSArray> boilerplate_array = Handle<JSArray>::cast(boilerplate);
1249 builder.Store( 1249 builder.Store(
1250 AccessBuilder::ForJSArrayLength(boilerplate_array->GetElementsKind()), 1250 AccessBuilder::ForJSArrayLength(boilerplate_array->GetElementsKind()),
1251 handle(boilerplate_array->length(), isolate())); 1251 handle(boilerplate_array->length(), isolate()));
1252 } 1252 }
1253 for (auto const inobject_field : inobject_fields) { 1253 for (auto const& inobject_field : inobject_fields) {
1254 builder.Store(inobject_field.first, inobject_field.second); 1254 builder.Store(inobject_field.first, inobject_field.second);
1255 } 1255 }
1256 return builder.Finish(); 1256 return builder.Finish();
1257 } 1257 }
1258 1258
1259 Node* JSCreateLowering::AllocateFastLiteralElements( 1259 Node* JSCreateLowering::AllocateFastLiteralElements(
1260 Node* effect, Node* control, Handle<JSObject> boilerplate, 1260 Node* effect, Node* control, Handle<JSObject> boilerplate,
1261 PretenureFlag pretenure, AllocationSiteUsageContext* site_context) { 1261 PretenureFlag pretenure, AllocationSiteUsageContext* site_context) {
1262 Handle<FixedArrayBase> boilerplate_elements(boilerplate->elements(), 1262 Handle<FixedArrayBase> boilerplate_elements(boilerplate->elements(),
1263 isolate()); 1263 isolate());
(...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
1378 return jsgraph()->simplified(); 1378 return jsgraph()->simplified();
1379 } 1379 }
1380 1380
1381 MachineOperatorBuilder* JSCreateLowering::machine() const { 1381 MachineOperatorBuilder* JSCreateLowering::machine() const {
1382 return jsgraph()->machine(); 1382 return jsgraph()->machine();
1383 } 1383 }
1384 1384
1385 } // namespace compiler 1385 } // namespace compiler
1386 } // namespace internal 1386 } // namespace internal
1387 } // namespace v8 1387 } // namespace v8
OLDNEW
« no previous file with comments | « src/compilation-statistics.cc ('k') | src/compiler/js-native-context-specialization.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698