Introduce DOTNET_JitReportMetrics

[Copilot]

Extracts JIT changes from #124943 into a standalone PR (per reviewer request) to prepare jitrollingbuilds with the new functionality first.

Description

JIT: DOTNET_JitReportMetrics config

Metrics.report(this) was called unconditionally in every DEBUG compilation, making O(metrics) JIT-EE calls per method even when no consumer needed them. Adds DOTNET_JitReportMetrics (default 0) to gate reporting uniformly across all build configurations; SuperPMI special-cases this config to always return 1.

• jitconfigvalues.h — adds RELEASE_CONFIG_INTEGER(JitReportMetrics, "JitReportMetrics", 0)
• jitmetadata.cpp — moves JitMetadata::report, reportValue, and JitMetrics::report out of #ifdef DEBUG so they compile in release builds (required since SuperPMI memorydiff uses release JIT binaries); dump/mergeToRoot remain debug-only
• compiler.cpp
  • Gates Metrics.BytesAllocated computation behind JitReportMetrics check (avoids arena page-walk overhead when metrics aren't consumed)
  • Gates Metrics.report(this) behind JitConfig.JitReportMetrics() — unified across DEBUG and release builds (no longer unconditional in DEBUG):

    if (JitConfig.JitReportMetrics())
    {
        Metrics.report(this);
    }

• superpmi/jithost.cpp — special-cases JitReportMetrics in JitHost::getIntConfigValue to always return 1, next to the existing SuperPMIMethodContextNumber special case
• superpmi-shim-collector/jithost.cpp — excludes JitReportMetrics from collection (same pattern as SuperPMIMethodContextNumber) so it is not recorded into method contexts

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[dotnet-policy-service]

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

[EgorBo]

Extracted from #124943 to prepare jitrollingbuilds with the new functionality first

PTAL @jakobbotsch @dotnet/jit-contrib

[Copilot]

Pull request overview

This PR introduces a new JIT configuration switch (DOTNET_JitReportMetrics / JitReportMetrics) to optionally report per-method JIT metrics back to the EE, and refactors metrics reporting so it can be enabled outside of DEBUG builds.

Changes:

• Added RELEASE_CONFIG_INTEGER(JitReportMetrics, "JitReportMetrics", 0) to enable config-controlled metrics reporting.
• Made JitMetadata::report, reportValue, and JitMetrics::report available outside #ifdef DEBUG so they can be used in retail builds.
• Updated Compiler::compCompileFinish() to call Metrics.report(this) only when JitReportMetrics is enabled, and moved Metrics.BytesAllocated computation out of #if MEASURE_MEM_ALLOC.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
src/coreclr/jit/jitmetadata.cpp	Moves core metric-reporting helpers out of #ifdef DEBUG while keeping debug-only helpers guarded.
src/coreclr/jit/jitconfigvalues.h	Adds the new JitReportMetrics release config flag.
src/coreclr/jit/compiler.cpp	Gates Metrics.report(this) behind JitReportMetrics and adjusts where BytesAllocated is computed.

[EgorBo]

I have no idea how/why this happened

[EgorBo]

@copilot please revert changes in ec15521 they're not related to this PR

[Copilot]

@copilot please revert changes in ec15521 they're not related to this PR

Reverted all 5 rollup-related files from ec15521 in commit 70de763. The PR now only contains the JIT changes.

[EgorBo]

No diffs

[jakobbotsch]

@copilot Can you please special case JitReportMetrics in JitHost::getIntConfigValue to always return 1 for SuperPMI, next to the special case for SuperPMIMethodContextNumber? Then unify the JIT side to always base the reporting of metrics on this config value, instead of making it dependent on DEBUG.

[Copilot]

@copilot Can you please special case JitReportMetrics in JitHost::getIntConfigValue to always return 1 for SuperPMI, next to the special case for SuperPMIMethodContextNumber? The...

Done in 5e64188. Added JitReportMetrics special case in SuperPMI's getIntConfigValue (returns 1, next to SuperPMIMethodContextNumber), added it to the shim-collector skip list, and unified the JIT side to always gate on JitConfig.JitReportMetrics() — no more DEBUG-specific unconditional path. Builds succeed in Debug, Checked, and Release.