Request for reviews (M): 7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
john.r.rose at oracle.com
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.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hotspot-compiler-dev