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

Side by Side Diff: src/lithium-allocator.cc

Issue 924453002: Fix invalid use of int in Zone. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Windows again... Created 5 years, 10 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 unified diff | Download patch
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 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/v8.h" 5 #include "src/v8.h"
6 6
7 #include "src/hydrogen.h" 7 #include "src/hydrogen.h"
8 #include "src/lithium-inl.h" 8 #include "src/lithium-inl.h"
9 #include "src/lithium-allocator-inl.h" 9 #include "src/lithium-allocator-inl.h"
10 #include "src/string-stream.h" 10 #include "src/string-stream.h"
(...skipping 2156 matching lines...) Expand 10 before | Expand all | Expand 10 after
2167 allocator_(allocator) { 2167 allocator_(allocator) {
2168 if (FLAG_hydrogen_stats) { 2168 if (FLAG_hydrogen_stats) {
2169 allocator_zone_start_allocation_size_ = 2169 allocator_zone_start_allocation_size_ =
2170 allocator->zone()->allocation_size(); 2170 allocator->zone()->allocation_size();
2171 } 2171 }
2172 } 2172 }
2173 2173
2174 2174
2175 LAllocatorPhase::~LAllocatorPhase() { 2175 LAllocatorPhase::~LAllocatorPhase() {
2176 if (FLAG_hydrogen_stats) { 2176 if (FLAG_hydrogen_stats) {
2177 unsigned size = allocator_->zone()->allocation_size() - 2177 size_t size = allocator_->zone()->allocation_size() -
2178 allocator_zone_start_allocation_size_; 2178 allocator_zone_start_allocation_size_;
2179 isolate()->GetHStatistics()->SaveTiming(name(), base::TimeDelta(), size); 2179 isolate()->GetHStatistics()->SaveTiming(name(), base::TimeDelta(), size);
2180 } 2180 }
2181 2181
2182 if (ShouldProduceTraceOutput()) { 2182 if (ShouldProduceTraceOutput()) {
2183 isolate()->GetHTracer()->TraceLithium(name(), allocator_->chunk()); 2183 isolate()->GetHTracer()->TraceLithium(name(), allocator_->chunk());
2184 isolate()->GetHTracer()->TraceLiveRanges(name(), allocator_); 2184 isolate()->GetHTracer()->TraceLiveRanges(name(), allocator_);
2185 } 2185 }
2186 2186
2187 #ifdef DEBUG 2187 #ifdef DEBUG
2188 if (allocator_ != NULL) allocator_->Verify(); 2188 if (allocator_ != NULL) allocator_->Verify();
2189 #endif 2189 #endif
2190 } 2190 }
2191 2191
2192 2192
2193 } } // namespace v8::internal 2193 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/lithium-allocator.h ('k') | src/typing.h » ('j') | src/zone.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698