[PATCH] D97504: [llvm-exegesis] Disable the LBR check on AMD

[Michał Górny via Phabricator via llvm-commits]

mgorny added inline comments.


================
Comment at: llvm/tools/llvm-exegesis/lib/X86/Target.cpp:735
+    // be confuse and think the AMD machine actually has LBR support.
+    if (state.getSubtargetInfo().getCPU().compare_lower("amd") != 0)
+      // If the kernel supports it, the hardware still may not have it.
----------------
I wonder if it wouldn't be better to explicitly check for Intel here, or maybe even for Intel family new enough for LBR. I'm going to guess that Cyrix/VIA processors don't have it either.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97504