[turbofan] Deferred blocks splintering.

src/compiler/live-range-separator.h

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_PREPROCESS_LIVE_RANGES_H_

[Benedikt Meurer, 2015/08/24 04:33:26]

Nit: Rename header guard.

[Mircea Trofin, 2015/08/24 20:55:10]

Done.

 #define V8_PREPROCESS_LIVE_RANGES_H_
 
 #include "src/compiler/register-allocator.h"

[Benedikt Meurer, 2015/08/24 04:33:26]

Don't include the header here, but forward declare Zone and
RegisterAllocationData below.

[Mircea Trofin, 2015/08/24 20:55:10]

Done.

 
 namespace v8 {
 namespace internal {
 namespace compiler {
 
 
-class PreprocessLiveRanges final {
+// A register allocation pair of transformations: splinter and merge live ranges
+class LiveRangeSeparator final {
  public:
-  PreprocessLiveRanges(RegisterAllocationData* data, Zone* zone)
+  LiveRangeSeparator(RegisterAllocationData* data, Zone* zone)
       : data_(data), zone_(zone) {}
-  void PreprocessRanges();
+
+  void Splinter();
 
  private:
-  void SplitRangesAroundDeferredBlocks();
-
   RegisterAllocationData* data() { return data_; }

[Benedikt Meurer, 2015/08/24 04:33:26]

Nit: mark these accessors const.

[Mircea Trofin, 2015/08/24 20:55:10]

Done.

   Zone* zone() { return zone_; }
 
+  RegisterAllocationData* data_;

[Benedikt Meurer, 2015/08/24 04:33:26]

Nit: use const fields, i.e. RegisterAllocationData* const data_;

[Mircea Trofin, 2015/08/24 20:55:10]

Done.

+  Zone* zone_;
+};
+
+
+class LiveRangeMerger final {
+ public:
+  LiveRangeMerger(RegisterAllocationData* data, Zone* zone)
+      : data_(data), zone_(zone) {}
+
+  void Merge();
+
+ private:
+  RegisterAllocationData* data() { return data_; }

[Benedikt Meurer, 2015/08/24 04:33:26]

See above.

[Mircea Trofin, 2015/08/24 20:55:10]

Done.

+  Zone* zone() { return zone_; }
+
   RegisterAllocationData* data_;

[Benedikt Meurer, 2015/08/24 04:33:26]

See above.

[Mircea Trofin, 2015/08/24 20:55:10]

Done.

   Zone* zone_;
 };
 
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8
 #endif  // V8_PREPROCESS_LIVE_RANGES_H_