RFR(S) 8194271: jaotc crashes with --debug flag

Igor Veresov igor.veresov at oracle.com
Wed Jan 3 19:50:44 UTC 2018

That’s a good point. We should. I added a simple test:
http://cr.openjdk.java.net/~iveresov/8194271/webrev.01/ <http://cr.openjdk.java.net/~iveresov/8194271/webrev.01/>


> On Jan 3, 2018, at 1:13 AM, Tobias Hartmann <tobias.hartmann at oracle.com> wrote:
> Hi Igor,
> the fix looks good to me. Should we add --debug to our testing?
> Best regards,
> Tobias
> On 03.01.2018 07:02, Igor Veresov wrote:
>> The change fixes the crash that occurs when jaotc tries to print method disassembly. The problem is that HotSpotCompiledCodeBuilder.createCompiledCode() assumes that if its method argument is null it’s a stub. And stub always has a name. The absence of a name caused the crash later on in the C++ code in CodeInstaller::install().
>> The solution is to pass method name to HotSpotCompiledCodeBuilder.createCompiledCode(). I also strengthened the C++ to throw an exception instead of crashing if a stub doesn’t have a name.
>> Webrev: http://cr.openjdk.java.net/~iveresov/8194271/webrev.00/
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8194271
>> Thanks,
>> igor

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20180103/3f896c7a/attachment-0001.html>

More information about the hotspot-compiler-dev mailing list