RFR (XS) 8023976: assert(!CompilationPolicy::can_be_compiled(this, comp_level)) failed: sanity check
vladimir.x.ivanov at oracle.com
Thu Aug 29 11:31:25 PDT 2013
Vladimir, Chris, thank you for review.
Couldn't agree more...
Do we already have a RFE for cleanup of this part?
On 8/29/13 10:11 PM, Christian Thalinger wrote:
> Yes, looks good and yes, it's a mess. -- Chris
> On Aug 29, 2013, at 9:37 AM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>> The fix looks good. But this code is a mess.
>> On 8/29/13 8:33 AM, Vladimir Ivanov wrote:
>>> 18 lines changed: 16 ins; 0 del; 2 mod;
>>> This assertion (part of 8022832) doesn't hold for method handle intrinsics (MethodHandle::linkTo*): they should be
>>> always "compilable", even if a method is marked as not compilable - there's a special case in Method::is_not_compilable.
>>> These methods can be marked as not compilable when, for example, -XX:CompileOnly command is used.
>>> There is a discrepancy between Method::is_not_c_compilable() and Method::is_not_compilable(). Instead of changing
>>> the assertion, I try to avoid marking MH intrinsics as not compilable.
>>> Testing: failing test, HS regression tests.
>>> Best regards,
>>> Vladimir Ivanov
More information about the hotspot-compiler-dev