 Chromium Code Reviews
 Chromium Code Reviews Issue 1641653005:
  [turbofan] Frame elision only for stubs.  (Closed) 
  Base URL: https://chromium.googlesource.com/v8/v8.git@master
    
  
    Issue 1641653005:
  [turbofan] Frame elision only for stubs.  (Closed) 
  Base URL: https://chromium.googlesource.com/v8/v8.git@master| OLD | NEW | 
|---|---|
| 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 1423 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1434 Run<AllocateDoubleRegistersPhase<GreedyAllocator>>(); | 1434 Run<AllocateDoubleRegistersPhase<GreedyAllocator>>(); | 
| 1435 } else { | 1435 } else { | 
| 1436 Run<AllocateGeneralRegistersPhase<LinearScanAllocator>>(); | 1436 Run<AllocateGeneralRegistersPhase<LinearScanAllocator>>(); | 
| 1437 Run<AllocateDoubleRegistersPhase<LinearScanAllocator>>(); | 1437 Run<AllocateDoubleRegistersPhase<LinearScanAllocator>>(); | 
| 1438 } | 1438 } | 
| 1439 | 1439 | 
| 1440 if (FLAG_turbo_preprocess_ranges) { | 1440 if (FLAG_turbo_preprocess_ranges) { | 
| 1441 Run<MergeSplintersPhase>(); | 1441 Run<MergeSplintersPhase>(); | 
| 1442 } | 1442 } | 
| 1443 | 1443 | 
| 1444 if (FLAG_turbo_frame_elision) { | 1444 // We plan to enable frame elision only for stubs and bytecode handlers. | 
| 1445 if (FLAG_turbo_frame_elision && info()->IsStub()) { | |
| 
Mircea Trofin
2016/01/28 02:39:10
Is there a way to check this is a bytecode handler
 | |
| 1445 Run<LocateSpillSlotsPhase>(); | 1446 Run<LocateSpillSlotsPhase>(); | 
| 1446 Run<FrameElisionPhase>(); | 1447 Run<FrameElisionPhase>(); | 
| 1447 } | 1448 } | 
| 1448 | 1449 | 
| 1449 Run<AssignSpillSlotsPhase>(); | 1450 Run<AssignSpillSlotsPhase>(); | 
| 1450 | 1451 | 
| 1451 Run<CommitAssignmentPhase>(); | 1452 Run<CommitAssignmentPhase>(); | 
| 1452 Run<PopulateReferenceMapsPhase>(); | 1453 Run<PopulateReferenceMapsPhase>(); | 
| 1453 Run<ConnectRangesPhase>(); | 1454 Run<ConnectRangesPhase>(); | 
| 1454 Run<ResolveControlFlowPhase>(); | 1455 Run<ResolveControlFlowPhase>(); | 
| (...skipping 18 matching lines...) Expand all Loading... | |
| 1473 tcf << AsC1VRegisterAllocationData("CodeGen", | 1474 tcf << AsC1VRegisterAllocationData("CodeGen", | 
| 1474 data->register_allocation_data()); | 1475 data->register_allocation_data()); | 
| 1475 } | 1476 } | 
| 1476 | 1477 | 
| 1477 data->DeleteRegisterAllocationZone(); | 1478 data->DeleteRegisterAllocationZone(); | 
| 1478 } | 1479 } | 
| 1479 | 1480 | 
| 1480 } // namespace compiler | 1481 } // namespace compiler | 
| 1481 } // namespace internal | 1482 } // namespace internal | 
| 1482 } // namespace v8 | 1483 } // namespace v8 | 
| OLD | NEW |