You already have 'result' variable for argument(0), why reread?:


Usually IfTrue is the fast path. IGVN will revert that check to have that. But it is better to have it in code. Current code looks like double negation.

There is no explanation in changes why false_count never seen (== 0) is more important than true_count == 0. The comment says "one of the values".


On 3/16/15 11:26 AM, Vladimir Ivanov wrote:

MethodHandleImpl::profileBoolean doesn't update never-taken branch count
when hitting a deopt on it. As a result, for rarely taken branches
consequent compilations consider them as never-taken and prune them
again causing repeated deopts. It severely affects peak performance.

The fix is to update MHI::profileBoolean intrinsic to insert a guard and
uncommon trap w/ reexecute bit set for never-seen value. Once previously
never seen value is encountered, the execution resumes after deopt in
MHI::profileBoolean and corresponding count becomes non-zero.

The guard doesn't add any additional overhead, since it dominates all
value usages and all branches on the same value are eliminated.

Testing: java/lang/invoke, nashorn, octane

Best regards,
Vladimir Ivanov
mlvm-dev mailing list
mlvm-dev mailing list

Reply via email to