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

Unified Diff: src/lithium-allocator.h

Issue 17553004: Fix zone usage within LAllocator. (Closed) Base URL: git@github.com:v8/v8.git@master
Patch Set: Rebased on https://codereview.chromium.org/18022002 Created 7 years, 6 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
« no previous file with comments | « no previous file | src/lithium-allocator.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/lithium-allocator.h
diff --git a/src/lithium-allocator.h b/src/lithium-allocator.h
index 3f1ce90eb8006e0c313b8199de90580d6a456a14..e5edd3cf039505a408e71dbe47da6ee1d0f7d498 100644
--- a/src/lithium-allocator.h
+++ b/src/lithium-allocator.h
@@ -434,7 +434,7 @@ class LAllocator BASE_EMBEDDED {
LPlatformChunk* chunk() const { return chunk_; }
HGraph* graph() const { return graph_; }
Isolate* isolate() const { return graph_->isolate(); }
- Zone* zone() const { return zone_; }
+ Zone* zone() { return &zone_; }
int GetVirtualRegister() {
if (next_virtual_register_ >= LUnallocated::kMaxVirtualRegisters) {
@@ -570,8 +570,7 @@ class LAllocator BASE_EMBEDDED {
inline void SetLiveRangeAssignedRegister(LiveRange* range,
int reg,
- RegisterKind register_kind,
- Zone* zone);
+ RegisterKind register_kind);
// Return parallel move that should be used to connect ranges split at the
// given position.
@@ -598,7 +597,7 @@ class LAllocator BASE_EMBEDDED {
inline LGap* GapAt(int index);
- Zone* zone_;
+ Zone zone_;
LPlatformChunk* chunk_;
@@ -647,13 +646,12 @@ class LAllocator BASE_EMBEDDED {
class LAllocatorPhase : public CompilationPhase {
public:
- LAllocatorPhase(const char* name, LAllocator* allocator)
- : CompilationPhase(name, allocator->graph()->info()),
- allocator_(allocator) { }
+ LAllocatorPhase(const char* name, LAllocator* allocator);
~LAllocatorPhase();
private:
LAllocator* allocator_;
+ unsigned allocator_zone_start_allocation_size_;
DISALLOW_COPY_AND_ASSIGN(LAllocatorPhase);
};
« no previous file with comments | « no previous file | src/lithium-allocator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698