[PATCH] D72032: [llvm-exegesis] Add pfm counters for Zen2 (znver2).

[Ganesh Gopalasubramanian via Phabricator via llvm-commits]

GGanesh added inline comments.


================
Comment at: llvm/lib/Target/X86/X86PfmCounters.td:231
+  let IssueCounters = [
+    // FIXME: In the current model, all three AGUs are considered the same,
+    // but only two are supposed to be able to compute load addresses.
----------------
Can you please help me to comprehend this better.
Zen2 has an AGU scheduler which services the AGU pipelines. 
So why are we considering only two will be able to compute load addresses?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72032/new/

https://reviews.llvm.org/D72032