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

[John Rose]

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...
URL: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20111018/049ce661/attachment.html