In D118023#3267271, @tra wrote:

The patch uses a @gmail.com email. Should I change it to JackAKirk <jack.kirk@codeplay.com> to match the sign-off email?

In D118023#3266987, @tra wrote:

If you are not sure if you have commit access, you probably do not. It must be explicitly granted:
https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access

As for landing the patch, I do it manually with git. Never tried it with arc, so I can't say much, except that it will not work until you have commit access.

In D118023#3266877, @tra wrote:

LGTM. Should I commit the patch on your behalf?

In D118023#3265601, @JackAKirk wrote:

Note that the test, llvm/test/CodeGen/NVPTX/wmma.py line 210, had the correct value already but didn't seem to cover the mistake.

It appears that we do not actually test for the correct number of registers used in an instruction argument, only for the correct instruction variant itself.
It should've been caught if we were to attempt assembling generated PTX with ptxas.

Note that the test, llvm/test/CodeGen/NVPTX/wmma.py line 210, had the correct value already but didn't seem to cover the mistake.

In D116673#3241721, @tra wrote:

In D116673#3237873, @JackAKirk wrote:

I can land the patch on your behalf. Are you OK to use the name/email in this patch or do you prefer to use a different email for the LLVM commit?

Thanks very much. Yes the name/email in the patch is fine.

Note for the future. It would make it easier to land the patch if it was submitted to phabricator as a git patch, according to the instructions here: https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface

Review tracker would then include info about relevant base commit, author, commit message, etc. and it can be easily applied with arc patch to my tree. Using plain diff requires a bit of extra manual work to transfer all of that from the review tracker to the git commit.

I can land the patch on your behalf. Are you OK to use the name/email in this patch or do you prefer to use a different email for the LLVM commit?

Thanks very much. Yes the name/email in the patch is fine.

I thought I should let you know that I do not have commit access to land the patch. I'm also happy to wait a little longer in case you think other interested parties might still chime in.

Made suggested change to naming convention.