Discussion:
RFR (T) 8215034: Remove old HOTSWAP conditionals
(too old to reply)
c***@oracle.com
2018-12-07 21:57:43 UTC
Permalink
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
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
Loading...