Review request (XXXS) 6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops

Steve Bohne Stephen.Bohne at Sun.COM
Thu Jan 31 08:05:25 PST 2008


Any review is appreciated, should be an easy one.

Fixed 6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops

http://j2se.east/~sbohne/webrev/6598190/

Due to Sun Studio 11 C++ compiler bug 6629277, there is a missing
destructor call for a temporary object in the case statement in
DepChange::ContextStream::next() in dependencies.cpp.  This causes
the oop referred to by the temp object to not get unregistered,
resulting in failed assert in UnhandledOops::clear_unhandled_oops().

The fix is to work around the compiler bug by adding curly brackets
around the affected clause in the case statement.


Reviewed by:

Fix verified (y/n): y

Verification testing:
   PRT fastdebug w/ -XX:+CheckUnhandledOops

Other testing:
   JPRT


Thanks,
Steve

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 6598190.diff.txt
Url: http://mail.openjdk.java.net/pipermail/hotspot-dev/attachments/20080131/72aa989b/attachment.txt 


More information about the hotspot-dev mailing list