RFR(L) 8001532: C2 node files refactoring

Vladimir Kozlov vladimir.kozlov at oracle.com
Mon Mar 17 22:20:27 UTC 2014


File names usually match base class name of ideal nodes. Please change:

constnode back to connode
bitsnode   --> countbitsnode
narrownode --> narrowptrnode
optonode   --> opaquenode

PartialSubtypeCheckNode class should be in new intrinsicnode file 
together with other similar classes from memnode files: StrIntrinsicNode 
and related, EncodeISOArrayNode.

ThreadLocalNode can be kept in connode because it is kind of a constant 
pointer value.

Put BinaryNode into movenode.hpp since it references cmove nodes.

constnode.hpp is included into callnode.hpp so you don't need to include 
it into files which have callnode.hpp included. Yes, we had it before 
but you are cleaning the code.

thanks,
Vladimir

On 3/17/14 6:51 AM, Morris Meyer wrote:
> Folks,
>
> Could I get a review for 8001532 - refactoring the old connode file in
> the C2 source tree?
>
> I've broken connode into bitsnode, castnode, constnode, convertnode,
> movenode, narrownode, optonode and threadnode.
>
> This change has been through JPRT.
>
>          --morris meyer
>
> JBS - https://bugs.openjdk.java.net/browse/JDK-8001532
> WEBREV - http://cr.openjdk.java.net/~morris/8001532.01


More information about the hotspot-compiler-dev mailing list