RFR (T) 8215034: Remove old HOTSWAP conditionals

Discussion:

c***@oracle.com

2018-12-07 21:57:43 UTC

I was in the area and found these few remaining conditionals.

Tested with tier1 on Oracle platforms.

open webrev at http://cr.openjdk.java.net/~coleenp/8215034.01/webrev
bug link https://bugs.openjdk.java.net/browse/JDK-8215034

Thanks,
Coleen

Daniel D. Daugherty

2018-12-07 22:04:35 UTC

Permalink

Post by c***@oracle.com
I was in the area and found these few remaining conditionals.
Tested with tier1 on Oracle platforms.
open webrev at http://cr.openjdk.java.net/~coleenp/8215034.01/webrev

src/hotspot/share/code/codeCache.cpp
    No comments.

src/hotspot/share/code/codeCache.hpp
    No comments.

src/hotspot/share/interpreter/templateInterpreter.hpp
    No comments.

src/hotspot/share/utilities/globalDefinitions.hpp
    No comments.

Thumbs up! Agree this is 'trivial'.

Dan

Post by c***@oracle.com
bug link https://bugs.openjdk.java.net/browse/JDK-8215034
Thanks,
Coleen

Jiangli Zhou

2018-12-07 22:06:14 UTC

Permalink

Looks good and trivial.

Thanks,

Jiangli

Post by c***@oracle.com
I was in the area and found these few remaining conditionals.
Tested with tier1 on Oracle platforms.
open webrev at http://cr.openjdk.java.net/~coleenp/8215034.01/webrev
bug link https://bugs.openjdk.java.net/browse/JDK-8215034
Thanks,
Coleen

c***@oracle.com

2018-12-07 22:50:49 UTC

Permalink

Thanks Dan, Jiangli, and Serguei!
Coleen

Hi Coleen,
+1
Thanks,
Serguei

Post by Jiangli Zhou
Looks good and trivial.
Thanks,
Jiangli