[PATCH] D40420: [CodeGen] Print "%vreg0" as "%0" in both MIR and debug output

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 28 11:01:56 PST 2017


MatzeB added a comment.

In https://reviews.llvm.org/D40420#937822, @qcolombet wrote:

> This is a personal taste, but I find %0 less obvious than %vreg0.
>  I.e., I would be against this change and I would rather change the dump of .mir.


I'd propose a compromise here:

1. Switch the debug printing to use the same code as the MIRPrinter today (%[0-9]+)
2. Once we have support for using arbitrary names for vregs we can choose any names when printing vregs. At that point change the debug printer to print %vreg[0-9]+


https://reviews.llvm.org/D40420





More information about the llvm-commits mailing list