[PATCH] D71927: [llvm-exegesis] Check counters before running

[Clement Courbet via Phabricator via llvm-commits]

courbet added inline comments.


================
Comment at: llvm/tools/llvm-exegesis/lib/Target.cpp:57
                                       const LLVMState &State) const {
+  PfmCountersInfo PfmCouters = State.getPfmCounters();
   switch (Mode) {
----------------
typo: `Couters`


================
Comment at: llvm/tools/llvm-exegesis/lib/Target.cpp:72
   case InstructionBenchmark::Uops:
+    if (!PfmCouters.UopsCounter)
+      report_fatal_error(
----------------
This is a change from the previous behaviour: The counter for number of uops is optional.


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

https://reviews.llvm.org/D71927