Skip to content
Success

Changes

Summary

  1. Create debug counters correctly for removed call sites (commit: be71530) (details)
  2. Specify the current compilation in failed call site debug counter names (commit: 2df7ed3) (details)
Commit be7153075cc18af5d5e3a167233c888a680f18d7 by Devin Papineau
Create debug counters correctly for removed call sites

This was passing callsite->_callNodeTreeTop while it was still null,
which prevented the debug counter increment from being generated and
even prevented a static debug counter increment.

The treetop where the call occurs is tt, and in this case callsite is
the result of searching for a dead call site that matches it. This tt
is already used for the Unresolved_In_ECS case, and it's also the value
that callsite->_callNodeTreeTop would take on just below if callsite
hadn't been removed.
(commit: be71530)
The file was modified compiler/optimizer/Inliner.cpp (diff)
Commit 2df7ed37ba5d69c098861e34292074c20e1aa2d2 by Devin Papineau
Specify the current compilation in failed call site debug counter names

This way, if a particular failure reason is of interest, the debug
counters will list all of the methods in which inlining failed for that
reason.
(commit: 2df7ed3)
The file was modified compiler/optimizer/Inliner.cpp (diff)