Skip to content

Conversation

@mahesh-attarde
Copy link
Contributor

@mahesh-attarde mahesh-attarde commented Jun 30, 2025

We added LNLP sched model recently, PFM counter bounding names needs to match cpu string.
llvm-exegesis wont produce results without correct naming.

@llvmbot
Copy link
Member

llvmbot commented Jun 30, 2025

@llvm/pr-subscribers-backend-x86

Author: Mahesh-Attarde (mahesh-attarde)

Changes

We added LNLP sched model recently, PFM counter bounding names needs to match cpu string.


Full diff: https://github.com/llvm/llvm-project/pull/146348.diff

1 Files Affected:

  • (modified) llvm/lib/Target/X86/X86PfmCounters.td (+1-1)
diff --git a/llvm/lib/Target/X86/X86PfmCounters.td b/llvm/lib/Target/X86/X86PfmCounters.td
index 20d96ffdb482e..b299633446410 100644
--- a/llvm/lib/Target/X86/X86PfmCounters.td
+++ b/llvm/lib/Target/X86/X86PfmCounters.td
@@ -254,7 +254,7 @@ def LunarLakePfmCounters : ProcPfmCounters {
   ];
   let ValidationCounters = DefaultIntelPfmValidationCounters;
 }
-def : PfmCountersBinding<"Lunarlake", LunarLakePfmCounters>;
+def : PfmCountersBinding<"lunarlake", LunarLakePfmCounters>;
 
 // AMD X86 Counters.
 defvar DefaultAMDPfmValidationCounters = [

Copy link
Collaborator

@RKSimon RKSimon left a comment

Choose a reason for hiding this comment

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

LGTM

@abhishek-kaushik22 abhishek-kaushik22 merged commit 84e8274 into llvm:main Jul 1, 2025
7 checks passed
@mahesh-attarde mahesh-attarde deleted the fix_exeg_pfm branch July 1, 2025 04:59
@mahesh-attarde
Copy link
Contributor Author

Thanks @abhishek-kaushik22

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants