[PATCH] D48209: [MCA] Add -summary-view option

[Clement Courbet via Phabricator via llvm-commits]

courbet added a comment.

In https://reviews.llvm.org/D48209#1133429, @andreadb wrote:

> Not sure what other reviewers think about this.
>  However, my opinion is that the summary view should never be optional in llvm-mca.
>  It is only a few lines, and it gives a nice overview of the run.


I mostly agree, but it does not hurt to have the option to disable it if you don;t want it to appear. Given that the patch is very small, I would be in favor of submitting.


Repository:
  rL LLVM

https://reviews.llvm.org/D48209