Skip to content

fix: add perf level for one apply of XSPerfAccumulate#140

Open
Yan-Muzi wants to merge 1 commit intomasterfrom
fix-perf-accu
Open

fix: add perf level for one apply of XSPerfAccumulate#140
Yan-Muzi wants to merge 1 commit intomasterfrom
fix-perf-accu

Conversation

@Yan-Muzi
Copy link
Contributor

@Yan-Muzi Yan-Muzi commented Mar 6, 2026

No description provided.

@Yan-Muzi Yan-Muzi requested review from Copilot and ngc7331 and removed request for Copilot March 6, 2026 07:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an optional perfLevel parameter to the XSPerfAccumulate.apply(perfNamePrefix, perfValid, perfCntSeq, ...) overload so callers can control the logging threshold instead of it always using XSPerfLevel.VERBOSE.

Changes:

  • Extend the XSPerfAccumulate “prefix + sequence” apply overload with a perfLevel argument (defaulting to VERBOSE).
  • Thread the provided perfLevel through to the underlying per-counter XSPerfAccumulate calls.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants