Skip to content

Commit 31df6a6

Browse files
committed
8283188: Build time regression caused by JDK-8278917
Reviewed-by: kbarrett, tschatzl
1 parent 0f1766d commit 31df6a6

File tree

3 files changed

+8
-9
lines changed

3 files changed

+8
-9
lines changed

src/hotspot/share/gc/g1/g1CollectedHeap.cpp

+7
Original file line numberDiff line numberDiff line change
@@ -3313,6 +3313,13 @@ HeapRegion* G1CollectedHeap::alloc_highest_free_region() {
33133313
return NULL;
33143314
}
33153315

3316+
void G1CollectedHeap::mark_evac_failure_object(const oop obj, uint worker_id) const {
3317+
// All objects failing evacuation are live. What we'll do is
3318+
// that we'll update the prev marking info so that they are
3319+
// all under PTAMS and explicitly marked.
3320+
_cm->par_mark_in_prev_bitmap(obj);
3321+
}
3322+
33163323
// Optimized nmethod scanning
33173324

33183325
class RegisterNMethodOopClosure: public OopClosure {

src/hotspot/share/gc/g1/g1CollectedHeap.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -1249,7 +1249,7 @@ class G1CollectedHeap : public CollectedHeap {
12491249
inline bool is_obj_dead_full(const oop obj) const;
12501250

12511251
// Mark the live object that failed evacuation in the prev bitmap.
1252-
inline void mark_evac_failure_object(const oop obj, uint worker_id) const;
1252+
void mark_evac_failure_object(const oop obj, uint worker_id) const;
12531253

12541254
G1ConcurrentMark* concurrent_mark() const { return _cm; }
12551255

src/hotspot/share/gc/g1/g1CollectedHeap.inline.hpp

-8
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@
2929

3030
#include "gc/g1/g1BarrierSet.hpp"
3131
#include "gc/g1/g1CollectorState.hpp"
32-
#include "gc/g1/g1ConcurrentMark.inline.hpp"
3332
#include "gc/g1/g1EvacFailureRegions.hpp"
3433
#include "gc/g1/g1Policy.hpp"
3534
#include "gc/g1/g1RemSet.hpp"
@@ -234,13 +233,6 @@ inline bool G1CollectedHeap::is_obj_dead_full(const oop obj) const {
234233
return is_obj_dead_full(obj, heap_region_containing(obj));
235234
}
236235

237-
inline void G1CollectedHeap::mark_evac_failure_object(const oop obj, uint worker_id) const {
238-
// All objects failing evacuation are live. What we'll do is
239-
// that we'll update the prev marking info so that they are
240-
// all under PTAMS and explicitly marked.
241-
_cm->par_mark_in_prev_bitmap(obj);
242-
}
243-
244236
inline void G1CollectedHeap::set_humongous_reclaim_candidate(uint region, bool value) {
245237
assert(_hrm.at(region)->is_starts_humongous(), "Must start a humongous object");
246238
_humongous_reclaim_candidates.set_candidate(region, value);

0 commit comments

Comments
 (0)