Skip to content

Commit

Permalink
Reenable internal pointers for <= warm opt level
Browse files Browse the repository at this point in the history
Internal pointers are never actually disabled for <= warm level until
recent refactoring in OMR changed it to use the correct API and disbled
it for real. Disabling internal pointers for <= warm opt level would
hurt performance so we need to enable it.

pr: eclipse-omr#2715

Signed-off-by: Yi Zhang <yizhang@ca.ibm.com>
  • Loading branch information
Yi Zhang committed Jul 30, 2018
1 parent 8bbe28c commit d9dfbdc
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions compiler/compile/OMRCompilation.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -919,15 +919,7 @@ int32_t OMR::Compilation::compile()
bool printCodegenTime = self()->getOption(TR_CummTiming);

if (self()->isOptServer())
{
// Temporarily exclude PPC due to perf regression
if( (self()->getMethodHotness() <= warm))
{
if (!TR::Compiler->target.cpu.isPower())
self()->getOptions()->setOption(TR_DisableInternalPointers);
}
self()->getOptions()->setOption(TR_DisablePartialInlining);
}

#ifdef J9_PROJECT_SPECIFIC
if (self()->getOptions()->getDelayCompile())
Expand Down

0 comments on commit d9dfbdc

Please sign in to comment.