[PATCH] D140273: [MC] Use `MCRegister` instead of `unsigned` in `MCTargetAsmParser`

Sergei Barannikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 18 11:51:18 PST 2022


barannikov88 added a comment.

In D140273#4003886 <https://reviews.llvm.org/D140273#4003886>, @MaskRay wrote:

> Perhaps fix that in a separate commit?

Fine by me.

In D140273#4003882 <https://reviews.llvm.org/D140273#4003882>, @MaskRay wrote:

> lib/Target/*/AsmParser and tablegen generated files still have many `unsigned RegNo` or `unsigned RegNum`, but this is a step forward...

I hoped no one would remember it. Those are less annoying though, so maybe next time.
Can I ask you to merge this patch? I still don't have permission.

"Sergei Barannikov <barannikov88 at gmail.com>".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140273



More information about the llvm-commits mailing list