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

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Mon Nov 9 19:16:35 UTC 2015

Still looks good.

Best regards,
Vladimir Ivanov

On 11/9/15 9:51 PM, Aleksey Shipilev wrote:
> Hi Severin,
> On 11/09/2015 08:57 PM, Severin Gehwolf wrote:
>> Here is a partial implementation of the Zero bits. In fact, the code
>> was already there to handle other getters too. I don't know why it was
>> guarded with those asserts and with is_simple_accessor().
>> It does not yet implement the setter logic, but at least we don't have
>> to keep is_simple_accessor() around. The patch builds on top of your
>> latest 8140650/webrev.02/ webrev.
>> The patch I'm talking about is:
>> http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8140650/JDK-8140650-is_accessor_zero_cleanup.hotspot.patch
> Excellent, thanks! Incorporated:
>    http://cr.openjdk.java.net/~shade/8140650/webrev.03/
> Coleen, are you still good with these changes?
> The RBT tests (hotspot_all, vm.{gc|regression|runtime}.testlist, and
> some others) came clean for a previous version, and this version touches
> only Zero code, so I am inclined to think this change does not need
> re-testing.
> Thanks,
> -Aleksey

More information about the hotspot-dev mailing list