RFR(S): 8028737 Remove JVM_AccessVMIntFlag / JVM_AccessVMBooleanFlag / JVM_VMBreakPoint

Daniel D. Daugherty daniel.daugherty at oracle.com
Mon Dec 16 06:22:12 PST 2013


On 12/16/13 3:35 AM, Staffan Larsen wrote:
> Dan, Coleen, David,
>
> Thanks all for looking at this code. Now that the 9 repos finally opened up and I tried pushing this I discovered that the change was incomplete. In addition to removing the code I also needed to update the mapfiles.
>
> Updated code: http://cr.openjdk.java.net/~sla/8028737/webrev.01/

Thumbs up.

make/bsd/makefiles/mapfile-vers-debug
make/linux/makefiles/mapfile-vers-debug
make/solaris/makefiles/mapfile-vers-debug
src/share/vm/prims/jvm.cpp
     No comments.

Dan


>
> Thanks,
> /Staffan
>
> On 20 nov 2013, at 21:53, Coleen Phillimore <coleen.phillimore at oracle.com> wrote:
>
>> Yes, this looks good.
>> Coleen
>>
>> On 11/20/2013 3:45 PM, Daniel D. Daugherty wrote:
>>> On 11/20/13 11:26 AM, Staffan Larsen wrote:
>>>> These are apparently old debugging code that is not referenced anymore.
>>>>
>>>> Targeted for jdk9, so I will hold on to (yet another) push until the repos open up.
>>>>
>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8028737
>>>> webrev: http://cr.openjdk.java.net/~sla/8028737/webrev.00/
>>> src/share/vm/prims/jvm.cpp
>>>     No comments.
>>>
>>> Thumbs up!
>>>
>>> Dan
>>>
>>>
>>>> Thanks,
>>>> /Staffan



More information about the hotspot-runtime-dev mailing list