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

Unified Diff: src/compiler/register-allocator.h

Issue 669053002: [turbofan] split compilation stats off from HStatistics and track high water marks (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: src/compiler/register-allocator.h
diff --git a/src/compiler/register-allocator.h b/src/compiler/register-allocator.h
index 60d08433af173a09941da10f3a416f88a1c493bb..3b6656ee8ff4d42ad89d17ecd11b8df921be5dd5 100644
--- a/src/compiler/register-allocator.h
+++ b/src/compiler/register-allocator.h
@@ -23,6 +23,8 @@ class PointerMap;
namespace compiler {
+class PipelineStatistics;
+
enum RegisterKind {
UNALLOCATED_REGISTERS,
GENERAL_REGISTERS,
@@ -331,8 +333,7 @@ class RegisterAllocator BASE_EMBEDDED {
// Returns the register kind required by the given virtual register.
RegisterKind RequiredRegisterKind(int virtual_register) const;
- // TODO(dcarney): fix compilation phase stats to not require this.
- bool Allocate(ZonePool* zone_pool = NULL);
+ bool Allocate(PipelineStatistics* stats = NULL);
const ZoneList<LiveRange*>* live_ranges() const { return &live_ranges_; }
const Vector<LiveRange*>* fixed_live_ranges() const {
@@ -344,7 +345,6 @@ class RegisterAllocator BASE_EMBEDDED {
CompilationInfo* info() const { return info_; }
inline InstructionSequence* code() const { return code_; }
- ZonePool* zone_pool() const { return zone_pool_; }
// This zone is for datastructures only needed during register allocation.
inline Zone* zone() const { return zone_; }
@@ -501,8 +501,6 @@ class RegisterAllocator BASE_EMBEDDED {
Frame* frame() const { return frame_; }
Zone* const zone_;
- // TODO(dcarney): remove this.
- ZonePool* zone_pool_;
Frame* const frame_;
CompilationInfo* const info_;
InstructionSequence* const code_;

Powered by Google App Engine
This is Rietveld 408576698