RFR 8213203: [JVMCI] adopt formatting changes from jvmci 8

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Nov 2 00:10:30 UTC 2018


After offline discussion and looking on old review I found that original complains were about overview.html files which 
were in some jvmci and graal directory. package-info.java files are fine and present in other JDK directories.

I am fine with these changes.

Thanks,
Vladimir

On 11/1/18 4:10 PM, Tom Rodriguez wrote:
> 
> 
> Vladimir Kozlov wrote on 11/1/18 1:51 PM:
>> Looks good except new package-info.java files. I don't think we need them in JDK. One time ago we removed them all.
>> I see we still have 2 files in JVMCI and 10  in Graal sources. We should remove them.
> 
> I'll go ahead and remove them.  We use them in graal-jvmci-8 to drive our sigtest logic which validates the public API 
> we expose and I figured it wouldn't hurt to bring them across.
> 
> tom
> 
>>
>> Thanks,
>> Vladimir
>>
>> On 11/1/18 1:21 PM, Tom Rodriguez wrote:
>>> http://cr.openjdk.java.net/~never/8213203/webrev
>>> https://bugs.openjdk.java.net/browse/JDK-8213203
>>>
>>> This brings over primarily formatting changes from graal-jvmci-8 to partially synchronize the sources.  Mostly these 
>>> are documentation improvements and style cleanups.  There are also some changes to reduce the visibility of 
>>> non-public classes in the public API. Testing is underway.
>>>
>>> tom


More information about the hotspot-compiler-dev mailing list