Skip to content

Commit

Permalink
Remove the createOrClonedNode function in OMR
Browse files Browse the repository at this point in the history
TR::Node *createOrFindClonedNode function has only relevance to the
OpenJ9 project. Therefore, remove it from OMR and relocate it to the
J9::CodeGenerator instead.

Signed-off-by: Bohao(Aaron) Wang <[email protected]>
  • Loading branch information
wbh123456 committed May 16, 2019
1 parent 2d1e785 commit 1041be8
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 23 deletions.
21 changes: 0 additions & 21 deletions compiler/codegen/OMRCodeGenerator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1293,27 +1293,6 @@ bool OMR::CodeGenerator::areAssignableGPRsScarce()
return (self()->getMaximumNumbersOfAssignableGPRs() <= threshold);
}

// J9
//
TR::Node *
OMR::CodeGenerator::createOrFindClonedNode(TR::Node *node, int32_t numChildren)
{
TR_HashId index;
if (!_uncommmonedNodes.locate(node->getGlobalIndex(), index))
{
// has not been uncommoned already, clone and store for later
TR::Node *clone = TR::Node::copy(node, numChildren);
_uncommmonedNodes.add(node->getGlobalIndex(), index, clone);
node = clone;
}
else
{
// found previously cloned node
node = (TR::Node *) _uncommmonedNodes.getData(index);
}
return node;
}


// returns true iff, based on opcodes and offsets if two store/load/address nodes are definitely disjoint (i.e. guaranteed not to overlap)
// any combination of stores,loads and address nodes are allowed as node1 and node2
Expand Down
2 changes: 0 additions & 2 deletions compiler/codegen/OMRCodeGenerator.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -1391,8 +1391,6 @@ class OMR_EXTENSIBLE CodeGenerator

// --------------------------------------------------------------------------

TR::Node *createOrFindClonedNode(TR::Node *node, int32_t numChildren);

bool constantAddressesCanChangeSize(TR::Node *node);
bool profiledPointersRequireRelocation();
bool needGuardSitesEvenWhenGuardRemoved();
Expand Down

0 comments on commit 1041be8

Please sign in to comment.