[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