[14] RFR(XS): 8231058: VerifyOops crashes with assert(_offset >= 0) failed: offset for non comment?

Tobias Hartmann tobias.hartmann at oracle.com
Tue Sep 17 05:20:08 UTC 2019


I've noticed that -XX:+VerifyOops triggers an unrelated assert failure on Sparc (probably a false
positive similar to JDK-8200288). I've filed JDK-8231100 and disabled the test on Sparc for now.

I've updated the webrev in-place:


On 16.09.19 16:19, Tobias Hartmann wrote:
> Hi,
> please review the following patch:
> https://bugs.openjdk.java.net/browse/JDK-8231058
> http://cr.openjdk.java.net/~thartmann/8231058/webrev.00/
> I've accidentally broke CodeStrings::find_last with my patch for JDK-8224624 [1] but didn't notice
> because the failure only triggers with -XX:+VerifyOops and apparently we don't have any tests that
> execute with that flag.
> The problem is that CodeString::offset() should only be called for comments. I've modified the
> condition accordingly. I've verified that the behavior is now equivalent to before JDK-8224624.
> Thanks,
> Tobias
> [1] https://bugs.openjdk.java.net/browse/JDK-8224624
> http://cr.openjdk.java.net/~thartmann/8224624/webrev.00/src/hotspot/share/asm/codeBuffer.cpp.sdiff.html

More information about the hotspot-dev mailing list