Request for reviews (M): 7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop

John Rose john.r.rose at
Tue Oct 18 14:49:20 PDT 2011

On Oct 18, 2011, at 2:16 PM, Vladimir Kozlov wrote:

> I don't trust C++ compilers to eliminate is_interpreted_frame() call in product VM even if the body of 'if' statement is empty. You can enclose it with #ifdef ASSERT if you want to keep 'if'.

Right; that's what I would do.  At some point the body of the assert gets so hairy that you have to break down and write #ifdef ASSERT, to contain the debug code.

-- John
-------------- next part --------------
An HTML attachment was scrubbed...

More information about the hotspot-compiler-dev mailing list