Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Swap ports 10 and 11 in Golden Cove Scheduler Models #117360

Open
boomanaiden154 opened this issue Nov 22, 2024 · 0 comments
Open

Swap ports 10 and 11 in Golden Cove Scheduler Models #117360

boomanaiden154 opened this issue Nov 22, 2024 · 0 comments
Assignees
Labels
backend:X86 Scheduler Models Accuracy of X86 scheduler models

Comments

@boomanaiden154
Copy link
Contributor

The documentation is incorrect according to intel/perfmon#149. We should update our scheduling models to be consistent with all the other tools.

@boomanaiden154 boomanaiden154 self-assigned this Nov 22, 2024
@boomanaiden154 boomanaiden154 added the backend:X86 Scheduler Models Accuracy of X86 scheduler models label Nov 22, 2024
@RKSimon RKSimon changed the title Swap ports 10 and 11 in Golden Cover Scheduler Models Swap ports 10 and 11 in Golden Cove Scheduler Models Nov 23, 2024
boomanaiden154 added a commit to boomanaiden154/llvm-project that referenced this issue Nov 24, 2024
Based on intel/perfmon#149, the documentation is
incorrect and the pfm counter names are actually correct. This patch adjusts
the Alder Lake scheduling model to match the performance counter naming/
correct naming that will soon be reflected in the optimization manual.

This fixes part of llvm#117360.
boomanaiden154 added a commit to boomanaiden154/llvm-project that referenced this issue Nov 24, 2024
Based on intel/perfmon#149, the documentation is incorrect and the pfm counter
names are actually correct. This patch adjusts the SapphireRapids scheduling
model to match the performance counter naming/ correct naming that will soon be
reflected in the optimization manual.

This fixes part of llvm#117360.
boomanaiden154 added a commit to boomanaiden154/llvm-project that referenced this issue Nov 24, 2024
Based on intel/perfmon#149, the documentation is
incorrect and the pfm counter names are actually correct. This patch adjusts
the Alder Lake scheduling model to match the performance counter naming/
correct naming that will soon be reflected in the optimization manual.

This fixes part of llvm#117360.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend:X86 Scheduler Models Accuracy of X86 scheduler models
Projects
None yet
Development

No branches or pull requests

1 participant