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

Side by Side Diff: src/compiler/pipeline.cc

Issue 1505233003: [turbofan] Don't run graph verifier on scheduled graphs. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years 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/compiler/pipeline.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/compiler/pipeline.h" 5 #include "src/compiler/pipeline.h"
6 6
7 #include <fstream> // NOLINT(readability/streams) 7 #include <fstream> // NOLINT(readability/streams)
8 #include <sstream> 8 #include <sstream>
9 9
10 #include "src/base/adapters.h" 10 #include "src/base/adapters.h"
(...skipping 1221 matching lines...) Expand 10 before | Expand all | Expand 10 after
1232 if (json_file != nullptr) { 1232 if (json_file != nullptr) {
1233 OFStream json_of(json_file); 1233 OFStream json_of(json_file);
1234 json_of << "{\"function\":\"" << info.GetDebugName().get() 1234 json_of << "{\"function\":\"" << info.GetDebugName().get()
1235 << "\", \"source\":\"\",\n\"phases\":["; 1235 << "\", \"source\":\"\",\n\"phases\":[";
1236 fclose(json_file); 1236 fclose(json_file);
1237 } 1237 }
1238 } 1238 }
1239 1239
1240 Pipeline pipeline(&info); 1240 Pipeline pipeline(&info);
1241 pipeline.data_ = &data; 1241 pipeline.data_ = &data;
1242 pipeline.RunPrintAndVerify("Machine", true); 1242 DCHECK_NOT_NULL(data.schedule());
rmcilroy 2015/12/08 17:26:35 This change also means that --trace-turbo no longe
Michael Starzinger 2015/12/08 17:37:34 Done.
1243
1243 return pipeline.ScheduleAndGenerateCode(call_descriptor); 1244 return pipeline.ScheduleAndGenerateCode(call_descriptor);
1244 } 1245 }
1245 1246
1246 1247
1247 Handle<Code> Pipeline::GenerateCodeForTesting(CompilationInfo* info, 1248 Handle<Code> Pipeline::GenerateCodeForTesting(CompilationInfo* info,
1248 Graph* graph, 1249 Graph* graph,
1249 Schedule* schedule) { 1250 Schedule* schedule) {
1250 CallDescriptor* call_descriptor = 1251 CallDescriptor* call_descriptor =
1251 Linkage::ComputeIncoming(info->zone(), info); 1252 Linkage::ComputeIncoming(info->zone(), info);
1252 return GenerateCodeForTesting(info, call_descriptor, graph, schedule); 1253 return GenerateCodeForTesting(info, call_descriptor, graph, schedule);
(...skipping 224 matching lines...) Expand 10 before | Expand all | Expand 10 after
1477 tcf << AsC1VRegisterAllocationData("CodeGen", 1478 tcf << AsC1VRegisterAllocationData("CodeGen",
1478 data->register_allocation_data()); 1479 data->register_allocation_data());
1479 } 1480 }
1480 1481
1481 data->DeleteRegisterAllocationZone(); 1482 data->DeleteRegisterAllocationZone();
1482 } 1483 }
1483 1484
1484 } // namespace compiler 1485 } // namespace compiler
1485 } // namespace internal 1486 } // namespace internal
1486 } // namespace v8 1487 } // namespace v8
OLDNEW
« no previous file with comments | « src/compiler/pipeline.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698