Skip to content
Permalink
Browse files
8253002: Remove the unused SafePointNode::_oop_map field
Reviewed-by: thartmann
  • Loading branch information
JornVernee committed Sep 15, 2020
1 parent dafcf10 commit d219d8b9871be037815413e6f52bc85ebac8db2a
Showing 2 changed files with 0 additions and 8 deletions.
@@ -54,7 +54,6 @@ class BoxLockNode;
class LockNode;
class UnlockNode;
class JVMState;
class OopMap;
class State;
class StartNode;
class MachCallNode;
@@ -329,14 +328,12 @@ class SafePointNode : public MultiNode {
// A plain safepoint advertises no memory effects (NULL):
const TypePtr* adr_type = NULL)
: MultiNode( edges ),
_oop_map(NULL),
_jvms(jvms),
_adr_type(adr_type)
{
init_class_id(Class_SafePoint);
}

OopMap* _oop_map; // Array of OopMap info (8-bit char) for GC
JVMState* const _jvms; // Pointer to list of JVM State objects
const TypePtr* _adr_type; // What type of memory does this node produce?
ReplacedNodes _replaced_nodes; // During parsing: list of pair of nodes from calls to GraphKit::replace_in_map()
@@ -349,8 +346,6 @@ class SafePointNode : public MultiNode {
void set_jvms(JVMState* s) {
*(JVMState**)&_jvms = s; // override const attribute in the accessor
}
OopMap *oop_map() const { return _oop_map; }
void set_oop_map(OopMap *om) { _oop_map = om; }

private:
void verify_input(JVMState* jvms, uint idx) const {
@@ -1346,9 +1346,6 @@ MachNode *Matcher::match_sfpt( SafePointNode *sfpt ) {
assert((mcall == NULL) || (mcall->jvms() == NULL) ||
(mcall->jvms()->debug_start() + mcall->_jvmadj == mcall->tf()->domain()->cnt()), "");

// Move the OopMap
msfpt->_oop_map = sfpt->_oop_map;

// Add additional edges.
if (msfpt->mach_constant_base_node_input() != (uint)-1 && !msfpt->is_MachCallLeaf()) {
// For these calls we can not add MachConstantBase in expand(), as the

1 comment on commit d219d8b

@bridgekeeper
Copy link

@bridgekeeper bridgekeeper bot commented on d219d8b Sep 15, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.