8232176: Shenandoah: new assert in ShenandoahEvacuationTask is too strong

Reviewed-by: rkennke
This commit is contained in:
Aleksey Shipilev 2019-10-14 11:03:13 +02:00
parent 9de8d240ff
commit 67a89143dd
2 changed files with 1 additions and 3 deletions

View File

@ -964,7 +964,6 @@ private:
ShenandoahHeapRegion* r;
while ((r =_cs->claim_next()) != NULL) {
assert(r->has_live(), "Region " SIZE_FORMAT " should have been reclaimed early", r->region_number());
assert(r->is_conc_move_allowed(), "Region " SIZE_FORMAT " should be movable", r->region_number());
_sh->marked_object_iterate(r, &cl);
if (ShenandoahPacing) {

View File

@ -198,8 +198,7 @@ public:
// Macro-properties:
bool is_alloc_allowed() const { return is_empty() || is_regular() || _state == _pinned; }
bool is_conc_move_allowed() const { return is_regular() || _state == _cset; }
bool is_stw_move_allowed() const { return is_conc_move_allowed() || (ShenandoahHumongousMoves && _state == _humongous_start); }
bool is_stw_move_allowed() const { return is_regular() || _state == _cset || (ShenandoahHumongousMoves && _state == _humongous_start); }
RegionState state() const { return _state; }
int state_ordinal() const { return region_state_to_ordinal(_state); }