Skip to content

Commit

Permalink
Merge pull request #20127 from ymanton/simplifier-trivial-cond-branches
Browse files Browse the repository at this point in the history
Add extra Simplifier & gotoElim passes to warm/hot
  • Loading branch information
vijaysun-omr authored Sep 18, 2024
2 parents 17a3fe8 + 8d9a6bd commit e5af31d
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion runtime/compiler/optimizer/J9Optimizer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -345,6 +345,8 @@ static const OptimizationStrategy warmStrategyOpts[] =
#endif
{ OMR::localCSE, OMR::IfLoopsAndNotProfiling },
{ OMR::idiomRecognition, OMR::IfLoopsAndNotProfiling },
{ OMR::treeSimplification },
{ OMR::redundantGotoElimination, OMR::IfEnabledAndNotJitProfiling },
{ OMR::blockSplitter },
{ OMR::treeSimplification }, // revisit; not really required ?
{ OMR::virtualGuardHeadMerger },
Expand Down Expand Up @@ -554,8 +556,9 @@ const OptimizationStrategy hotStrategyOpts[] =
{ OMR::earlyLocalGroup },
{ OMR::stripMiningGroup, OMR::IfLoops }, // strip mining in loops
{ OMR::loopReplicator, OMR::IfLoops }, // tail-duplication in loops
{ OMR::blockSplitter, OMR::IfNews }, // treeSimplification + blockSplitter + VP => opportunity for EA
{ OMR::expensiveGlobalValuePropagationGroup },
{ OMR::redundantGotoElimination, OMR::IfEnabledAndNotJitProfiling },
{ OMR::blockSplitter, OMR::IfNews }, // treeSimplification + blockSplitter + VP => opportunity for EA
{ OMR::localCSE, OMR::IfVectorAPI },
{ OMR::loopCanonicalization, OMR::IfVectorAPI },
{ OMR::partialRedundancyEliminationGroup, OMR::IfVectorAPI },
Expand Down

0 comments on commit e5af31d

Please sign in to comment.