AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

Fixed minor error in IncrementalMarking::AdvanceIncrementalMarkingOnAllocation

Browse Source 
bytes_processed should be used instead of bytes_to_process when
updating bytes_allocated_. Also updated the type of bytes_processed
since min() takes in 2 variables of the same type.
The code was added as part of:
https://codereview.chromium.org/2359903002

R=hpayer@chromium.org,ulan@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2380853002
Cr-Commit-Position: refs/heads/master@{#39857}
This commit is contained in:
bjaideep 2016-09-29 03:22:26 -07:00 committed by Commit bot
parent 964f54ba1a
commit 1fb4498c40
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/heap/incremental-marking.cc
View File

@ -1105,7 +1105,7 @@ void IncrementalMarking::AdvanceIncrementalMarkingOnAllocation() {
heap()->tracer()->IncrementalMarkingSpeedInBytesPerMillisecond());
bytes_to_process = Min(bytes_to_process, max_step_size);
intptr_t bytes_processed = 0;
size_t bytes_processed = 0;
if (bytes_marked_ahead_of_schedule_ >= bytes_to_process) {
// Steps performed in tasks have put us ahead of schedule.
// We skip processing of marking dequeue here and thus
@ -1116,7 +1116,7 @@ void IncrementalMarking::AdvanceIncrementalMarkingOnAllocation() {
bytes_processed = Step(bytes_to_process, GC_VIA_STACK_GUARD,
FORCE_COMPLETION, StepOrigin::kV8);
}
bytes_allocated_ -= Min(bytes_allocated_, bytes_to_process);
bytes_allocated_ -= Min(bytes_allocated_, bytes_processed);
}
}