RFR(S): 8077618: Move rtmLocking.cpp to shared directory.
Vladimir Kozlov
vladimir.kozlov at oracle.com
Tue Apr 14 02:09:28 UTC 2015
I updated copyright dates as David said and I submitted it to JPRT to
push since we have 2 reviews.
Thanks,
Vladimir
On 4/13/15 6:36 PM, David Holmes wrote:
> Hi Goetz,
>
> On 14/04/2015 1:57 AM, Lindenmaier, Goetz wrote:
>> Hi,
>>
>> we implemented support for Transactional Memory on ppc. For this, we
>> need
>> the file rtmLocking.cpp from subdirectory x86 as is.
>> Therefore I would like to move the file to share/vm/runtime, where the
>> corresponding header is placed. I added a guard around the code in
>> this file.
>
> Not sure if we need the guard in the cpp file - I guess it speeds up
> compilation a tiny amount :) If we do want it then the copyright date
> needs updating to "2014, 2015,".
>
>> Also, I moved #ifdef x86 from globalDefinitions.hpp to
>> globalDefinitions_x86.hpp
>> which simplifies enabling RTM on PPC.
>
> Copyright date in src/cpu/x86/vm/globalDefinitions_x86.hpp needs updating.
>
>> Please review this change. I please need a sponsor.
>> http://cr.openjdk.java.net/~goetz/webrevs/8077618-mvRTM/webrev.01/
>
> Otherwise looks okay.
>
> David
>
>> Best regards,
>> Goetz.
>>
>>
>>
>>
More information about the hotspot-dev
mailing list