RFR (S) 8140650: Method::is_accessor should cover getters and setters for all types

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Thu Nov 5 16:59:27 UTC 2015

>> If there's an RFE filed and Zero maintainers promise to fix it in a
>> prompt manner, I'm fine with what you proposed.
> Yeah, that's the plan.
>> Also, I had an idea: why don't you move is_simple_accessor() from Method
>> to some Zero-specific location? I don't see any value in keeping it in
>> Method.
> That's an interesting trick, but I think it will cause more harm: shared
> interpreter.cpp would have to know about that Zero-specific method, or
> otherwise intercept "too wide" accessor shape and narrow it down before
> it reaches Zero. Therefore, I believe is_simple_accessor is the lesser evil.
Ok. Looks good then.

Best regards,
Vladimir Ivanov

More information about the hotspot-dev mailing list