Skip to content

Commit 688045c

Browse files
committed
(0.51) Revert "Merge pull request eclipse-openj9#21056 from vijaysun-omr/jprofiling1"
This reverts commit 8bbc086, reversing changes made to a134afa. Related: eclipse-openj9#21256
1 parent 63ad5e2 commit 688045c

File tree

1 file changed

+9
-26
lines changed

1 file changed

+9
-26
lines changed

runtime/compiler/optimizer/JProfilingValue.cpp

Lines changed: 9 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -317,23 +317,11 @@ void
317317
TR_JProfilingValue::lowerCalls()
318318
{
319319
TR::TreeTop *cursor = comp()->getStartTree();
320-
bool stopProfiling = false;
321320
TR_BitVector *backwardAnalyzedAddressNodesToCheck = new (comp()->trStackMemory()) TR_BitVector();
322321
while (cursor)
323322
{
324323
TR::Node * node = cursor->getNode();
325324
TR::TreeTop *nextTreeTop = cursor->getNextTreeTop();
326-
int32_t ipMax = comp()->maxInternalPointers()/2;
327-
static const char * ipl = feGetEnv("TR_ProfilingIPLimit");
328-
if (ipl)
329-
{
330-
static const int32_t ipLimit = atoi(ipl);
331-
ipMax = ipLimit;
332-
}
333-
334-
if (!stopProfiling && (comp()->getSymRefTab()->getNumInternalPointers() >= ipMax))
335-
stopProfiling = true;
336-
337325
if (node->isProfilingCode() &&
338326
node->getOpCodeValue() == TR::treetop &&
339327
node->getFirstChild()->getOpCode().isCall() &&
@@ -366,20 +354,15 @@ TR_JProfilingValue::lowerCalls()
366354
}
367355

368356
backwardAnalyzedAddressNodesToCheck->empty();
369-
370-
if (!stopProfiling)
371-
{
372-
TR::Node *child = node->getFirstChild();
373-
dumpOptDetails(comp(), "%s Replacing profiling placeholder n%dn with value profiling trees\n",
374-
optDetailString(), child->getGlobalIndex());
375-
// Extract the arguments and add the profiling trees
376-
TR::Node *value = child->getFirstChild();
377-
TR_AbstractHashTableProfilerInfo *table = (TR_AbstractHashTableProfilerInfo*) child->getSecondChild()->getAddress();
378-
bool needNullTest = comp()->getSymRefTab()->isNonHelper(child->getSymbolReference(), TR::SymbolReferenceTable::jProfileValueWithNullCHKSymbol);
379-
addProfilingTrees(comp(), cursor, value, table, needNullTest, true, trace());
380-
// Remove the original trees and continue from the tree after the profiling
381-
}
382-
357+
TR::Node *child = node->getFirstChild();
358+
dumpOptDetails(comp(), "%s Replacing profiling placeholder n%dn with value profiling trees\n",
359+
optDetailString(), child->getGlobalIndex());
360+
// Extract the arguments and add the profiling trees
361+
TR::Node *value = child->getFirstChild();
362+
TR_AbstractHashTableProfilerInfo *table = (TR_AbstractHashTableProfilerInfo*) child->getSecondChild()->getAddress();
363+
bool needNullTest = comp()->getSymRefTab()->isNonHelper(child->getSymbolReference(), TR::SymbolReferenceTable::jProfileValueWithNullCHKSymbol);
364+
addProfilingTrees(comp(), cursor, value, table, needNullTest, true, trace());
365+
// Remove the original trees and continue from the tree after the profiling
383366
TR::TransformUtil::removeTree(comp(), cursor);
384367
if (trace())
385368
comp()->dumpMethodTrees("After Adding Profiling Trees");

0 commit comments

Comments
 (0)