Request for reviews (M): 7069452: Cleanup NodeFlags

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Jul 22 11:06:55 PDT 2011


http://cr.openjdk.java.net/~kvn/7069452/webrev

Fixed 7069452: Cleanup NodeFlags

We are almost out of bits (16) for Node::NodeFlags. I removed flags which 
duplicate information in Node::NodeClasses.
is_Call() uses class_id check and is now valid only for ideal CallNode.
is_Goto() checks ideal_opcode().
is_Vector() is replaced with check for Vector,VectorLoad,VectorStore classes.
MachProjNode was added to class_id check to avoid calling Opcode() in many 
places. MulNode was removed from class_id since it was used only in one place.

I removed all logic associated with is_pc_relative flag. It was only checked in 
one place during long to short branch replacement. I replaced it with check in 
adlc parser - short branches should be defined only for a branch to a label, 
which means branch with PC relative offset. To relay on developer to set 
ins_pc_relative() was mistake since it could be used in wrong places. For 
example, it was specified for table jumps, calls and FastLock/FastUnlock.


More information about the hotspot-compiler-dev mailing list