RFR (S): 8180755: Remove use of bitMap.inline.hpp include from instanceKlass.hpp and c1_ValueSet.hpp

David Holmes david.holmes at oracle.com
Wed May 24 01:27:11 UTC 2017


On 24/05/2017 1:31 AM, Thomas Schatzl wrote:
> Hi Coleen,
> 
> On Tue, 2017-05-23 at 10:15 -0400, coleen.phillimore at oracle.com wrote:
>> http://cr.openjdk.java.net/~tschatzl/8180755/webrev.1/src/share/vm/oo
>> ps/generateOopMap.cpp.udiff.html
>>
>> I don't think this should include methodHandles.hpp (I see you've
>> rearranged it, can you try to remove it?)

Why should it be removed? The file uses things from methodHandles.

David

>>
>> The rest of this looks fine to me.
> 
>    Done, still passes build-only jprt. New webrevs:
> 
> http://cr.openjdk.java.net/~tschatzl/8180755/webrev.1_to_2 (diff)
> http://cr.openjdk.java.net/~tschatzl/8180755/webrev.2 (full)
> 
> Thanks,
>    Thomas
> 


More information about the hotspot-runtime-dev mailing list