[PATCH] D116673: [Clang][NVPTX]Add NVPTX intrinsics and builtins for CUDA PTX cvt sm80 instructions

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 13 13:38:16 PST 2022


tra added a comment.

In D116673#3237873 <https://reviews.llvm.org/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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116673



More information about the cfe-commits mailing list