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

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 1 17:03:30 PDT 2017


MatzeB added a comment.

In https://reviews.llvm.org/D31262#742311, @qcolombet wrote:

> In a nutshell, I am saying that we shouldn't conflict two goals: editing and printing. Printing should be full fledged, editing should be minimal.


Sounds like we are not in sync what we do with .mir... It is a tool to create llvm lit tests, isn't it? Printing at least today is mostly done via MachineFunction::print() etc. Sure we want to unify the formats used there and .mir in the long term. But printing with llc -stop-after is the main tool to get started writing a lit test and we should promoted/encourage nice syntax there.


Repository:
  rL LLVM

https://reviews.llvm.org/D31262





More information about the llvm-commits mailing list