RFR(S) 8204209: [Graal] Compilation fails during nmethod printing with "assert(bci == 0 || 0 <= bci && bci < code_size()) failed: illegal bci"

[Igor Veresov]

After discussing this with Tom, we decided that it would be a bad idea to change values of final static fields as this would break compatibility (since javac inlines these values). So, for now, we’d have to do re-mapping.

Webrev: http://cr.openjdk.java.net/~iveresov/8204209/webrev.01 <http://cr.openjdk.java.net/~iveresov/8204209/webrev.01>

Thanks,
igor

> On Jun 18, 2018, at 10:12 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> 
> CCing to runtime group.
> 
> Seems fine to me.
> 
> Thanks,
> Vladimir
> 
> On 6/18/18 6:28 PM, Igor Veresov wrote:
>> Make hotspot tolerate negative placeholder BCIs that are produced by Graal.
>> Should this fix be deemed acceptable I’ll backport it to graal-jvmci-8.
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8204209
>> Webrev: http://cr.openjdk.java.net/~iveresov/8204209/webrev/
>> Thanks,
>> igor

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20180621/8646488b/attachment.html>