[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