[PATCH] D48209: [MCA] Add -summary-view option
Greg Bedwell via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 15 05:40:05 PDT 2018
gbedwell added a comment.
In https://reviews.llvm.org/D48209#1133474, @lebedev.ri wrote:
> Actually make `ninja check-llvm-tools-llvm-mca` pass, as suggested/noted by @gbedwell
Thanks. Please could you also remove the ALL prefix. I'm just testing a local patch to update_mca_test_checks.py that would make this a hard error:
$ py update_mca_test_checks.py ../test/tools/llvm-mca/X86/option-all-views-1.s
Test: ../test/tools/llvm-mca/X86/option-all-views-1.s
error: unused prefixes: ['ALL']
Repository:
rL LLVM
https://reviews.llvm.org/D48209
More information about the llvm-commits
mailing list