RFR: 8156552: [JVMCI] remove final and stable field handling from ConstantReflectionProvider

Roland Schatz roland.schatz at oracle.com
Thu May 12 13:24:06 UTC 2016


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