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