[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