[PATCH] D48190: [MCA][x86][NFC] Add tests for -register-file-stats, -scheduler-stats

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 15 01:14:13 PDT 2018


lebedev.ri added inline comments.


================
Comment at: test/tools/llvm-mca/X86/register-file-statistics.s:45
+# SKLSER-NEXT:    Total number of mappings created:    2
+# SKLSER-NEXT:    Max number of mappings used:         2
+
----------------
RKSimon wrote:
> A lot of these prefixes look redundant?
Oh, indeed.

BTW, in this file, everything before `Register File statistics:` isn't expected to be here.
Can
```
Iterations:        100
Instructions:      100
Total Cycles:      51
Dispatch Width:    2
IPC:               1.96
Block RThroughput: 0.5
```
be named a "-basic-stats", and an option to control that output be added?
@andreadb does that sound reasonable?


Repository:
  rL LLVM

https://reviews.llvm.org/D48190





More information about the llvm-commits mailing list