stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8198270: Remove dangerous assert in HandleArea::oops_do()

Browse Source 
Reviewed-by: coleenp
This commit is contained in:
Per Lidén 2017-12-08 13:47:08 +01:00
parent 28f0646ef1
commit 168e8b2abe
1 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/hotspot/share/runtime/handles.cpp
View File

@ -1,5 +1,5 @@
/* /*
* Copyright (c) 1997, 2017, Oracle and/or its affiliates. All rights reserved. * Copyright (c) 1997, 2018, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
* *
* This code is free software; you can redistribute it and/or modify it * This code is free software; you can redistribute it and/or modify it
@ -97,10 +97,6 @@ static uintx chunk_oops_do(OopClosure* f, Chunk* chunk, char* chunk_top) {
// during GC phase 3, a handle may be a forward pointer that // during GC phase 3, a handle may be a forward pointer that
// is not yet valid, so loosen the assertion // is not yet valid, so loosen the assertion
while (bottom < top) { while (bottom < top) {
// This test can be moved up but for now check every oop.
assert(oopDesc::is_oop(*bottom, true), "handle should point to oop");
f->do_oop(bottom++); f->do_oop(bottom++);
} }
return handles_visited; return handles_visited;