[PATCH] D31262: MIParser/MIRPrinter: Compute block successors if not explicitely specified

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 4 13:45:19 PDT 2017


MatzeB added a comment.

In https://reviews.llvm.org/D31262#710545, @ab wrote:

> In https://reviews.llvm.org/D31262#710542, @qcolombet wrote:
>
> > I'd like the printer to always print everything. I don't think there is any value in having magic here.
>
>
> IMO this kind of magic is very useful: it makes output much easier to read, and easier to start from when writing tests.


I also think there is value in reducing the amount of information printed. In this specific case the information omitted is quite obvious IMO.


Repository:
  rL LLVM

https://reviews.llvm.org/D31262





More information about the llvm-commits mailing list