RFC: 8224599: Remove globals_ext.hpp

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed May 22 14:55:29 UTC 2019


I think we should clean up all places.  I vote for v02.

Thanks,
Vladimir

On 5/22/19 4:58 AM, Stefan Karlsson wrote:
> I realized that some of the globals_ext code had been placed in os_ext.hpp and not only in globals_ext.hpp.
> 
> If it's appropriate, I've included it in this RFC:
>   http://cr.openjdk.java.net/~stefank/8224599/webrev.02.delta/
>   http://cr.openjdk.java.net/~stefank/8224599/webrev.02/
> 
> If not, I can send out a new RFC for that part.
> 
> Thanks,
> StefanK
> 
> On 2019-05-22 13:46, Stefan Karlsson wrote:
>> Hi all,
>>
>> Is anyone using globals_ext.hpp? If not, I propose that we remove it and clean up the code using it.
>>
>> http://cr.openjdk.java.net/~stefank/8224599/webrev.01/
>> https://bugs.openjdk.java.net/browse/JDK-8224599
>>
>> Since globals_ext.hpp pulled in jvmFlag.hpp, I had to fix some includes and forward declarations.
>>
>> Thanks,
>> StefanK


More information about the hotspot-dev mailing list