Fix inliner size scaling based on number of calls

When scaling the estimated size of a candidate for inlining based on
frequency and the number of estimated calls nested in the candidate, the
calculation inadvertantly ignores the number of estimated calls and uses
a value derived from the frequency twice. This doesn't appear to be the
intention of the code at all.

Signed-off-by: Younes Manton <ymanton@ca.ibm.com>

Author: ymanton

runtime/compiler/optimizer/InlinerTempForJ9.cpp

@@ -4155,7 +4155,7 @@ int32_t TR_MultipleCallTargetInliner::scaleBasedOnFrequency(TR_CallTarget *callt
 
         float avgMethodSize = (float)size / (float)ecs->getNumOfEstimatedCalls();
         float numCallsFactor = (float)(avgMethodSize) / 110.0f;
-        numCallsFactor = std::max(factor, 0.1f);
+        numCallsFactor = std::max(numCallsFactor, 0.1f);
 
         if (size > 100) {
             size = (int)((float)size * factor * numCallsFactor);