RFR: 8156552: [JVMCI] remove final and stable field handling from ConstantReflectionProvider
Vladimir Kozlov
vladimir.kozlov at oracle.com
Thu May 12 16:02:58 UTC 2016
Still good.
Thanks,
Vladimir
On 5/12/16 6:24 AM, Roland Schatz wrote:
> Thanks for the reviews.
>
> Turns out I have missed a bit of code that should also be removed.
>
> I have uploaded an updated webrev:
> http://cr.openjdk.java.net/~rschatz/JDK-8156552/webrev.01/
>
> Compared to the previous webrev this also removes the special handling of implicit stable fields in
> HotSpotResolvedJavaFieldImpl.isStable, making its behavior consistent with the other flag query methods.
>
> Please review again.
> Sorry for the iteration overhead ;)
>
> - Roland
>
> On 05/11/2016 05:41 PM, Vladimir Kozlov wrote:
>> Looks good.
>>
>> Thanks,
>> Vladimir
>>
>> On 5/11/16 3:20 AM, Roland Schatz wrote:
>>> Hi,
>>>
>>> Please review:
>>> http://cr.openjdk.java.net/~rschatz/JDK-8156552/webrev.00/
>>> https://bugs.openjdk.java.net/browse/JDK-8156552
>>>
>>> This is pure removal of a few methods from the
>>> ConstantReflectionProvider interface, their implementation, and their
>>> tests.
>>> All jvmci jtreg tests pass.
>>>
>>> Thanks,
>>> Roland
>
More information about the hotspot-compiler-dev
mailing list