[PATCH] D60311: MIR printer should lowercase sub-register names to be in sync with parser?

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope added a comment.

I think it would be better to let the printer use the same convention as the specified in the definitions (as used in the code).
And then I think it is a bug in the parser, as it tries to be case insensitive by matching the parsed string against a lower case definition. It should also lower the parsed string if we want the parser to be case insensitive.

If we fix the parser instead, then we still would get debug printouts etc that matches the mixed case used in the C++ code. Or maybe I haven't understood the problem correctly.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60311/new/

https://reviews.llvm.org/D60311