Mercurial > hg > graal-jvmci-8
comparison src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp @ 20274:a8137787acfe
8047821: G1 Does not use the save_marks functionality as intended
Summary: Rename save_marks to record_top_and_timestamp and remove som unused but related methods
Reviewed-by: stefank, ehelin
author | mgerdin |
---|---|
date | Thu, 26 Jun 2014 10:00:00 +0200 |
parents | b7d24d2bc8be |
children | 2c6ef90f030a |
comparison
equal
deleted
inserted
replaced
20273:9441d22e429a | 20274:a8137787acfe |
---|---|
4359 if (retained_region != NULL && | 4359 if (retained_region != NULL && |
4360 !retained_region->in_collection_set() && | 4360 !retained_region->in_collection_set() && |
4361 !(retained_region->top() == retained_region->end()) && | 4361 !(retained_region->top() == retained_region->end()) && |
4362 !retained_region->is_empty() && | 4362 !retained_region->is_empty() && |
4363 !retained_region->isHumongous()) { | 4363 !retained_region->isHumongous()) { |
4364 retained_region->set_saved_mark(); | 4364 retained_region->record_top_and_timestamp(); |
4365 // The retained region was added to the old region set when it was | 4365 // The retained region was added to the old region set when it was |
4366 // retired. We have to remove it now, since we don't allow regions | 4366 // retired. We have to remove it now, since we don't allow regions |
4367 // we allocate to in the region sets. We'll re-add it later, when | 4367 // we allocate to in the region sets. We'll re-add it later, when |
4368 // it's retired again. | 4368 // it's retired again. |
4369 _old_set.remove(retained_region); | 4369 _old_set.remove(retained_region); |
6302 true /* do_expand */); | 6302 true /* do_expand */); |
6303 if (new_alloc_region != NULL) { | 6303 if (new_alloc_region != NULL) { |
6304 // We really only need to do this for old regions given that we | 6304 // We really only need to do this for old regions given that we |
6305 // should never scan survivors. But it doesn't hurt to do it | 6305 // should never scan survivors. But it doesn't hurt to do it |
6306 // for survivors too. | 6306 // for survivors too. |
6307 new_alloc_region->set_saved_mark(); | 6307 new_alloc_region->record_top_and_timestamp(); |
6308 if (survivor) { | 6308 if (survivor) { |
6309 new_alloc_region->set_survivor(); | 6309 new_alloc_region->set_survivor(); |
6310 _hr_printer.alloc(new_alloc_region, G1HRPrinter::Survivor); | 6310 _hr_printer.alloc(new_alloc_region, G1HRPrinter::Survivor); |
6311 } else { | 6311 } else { |
6312 _hr_printer.alloc(new_alloc_region, G1HRPrinter::Old); | 6312 _hr_printer.alloc(new_alloc_region, G1HRPrinter::Old); |