On Mon, 14 Sep 2020 20:51:34 GMT, Daniel D. Daugherty <dcu...@openjdk.org> wrote:
>> This code seems like something that doesn't belong here anymore. This code >> assumed synchronous scanning of oops in >> ObjectMonitor and scanning memory regions, and that's no longer the case >> with OopStorage. I think this assert should be >> removed. It exports some implementation detail of now completely unrelated >> code in order to do a very specific check. > > @fisk - please chime in here... I agree. Also, the assert becomes true somewhat obviously because of its first clause, which is already guaranteed because of its placement in the surrounding else-branch (unless something really weird happens). ------------- PR: https://git.openjdk.java.net/jdk/pull/135
