[PATCH] D11220: Correct lowering of memmove in NVPTX

Okwan Kwon okwank at nvidia.com
Wed Jul 15 10:22:37 PDT 2015


okwank added a subscriber: okwank.
okwank added a comment.

Hi,

My name is Okwan Kwon, and I have two comments.

1. Use getRawDest() and getRawSource() instead. getDest() and getSource() will strip off any casts to give the original pointer. When the original pointer type is not i8 *, then it will get an assertion.

2. Can you find a way to make use of the alignment information from the memmove intrinsic? It might be possible to generate more efficient load/store than byte copying.

Okwan


Repository:
  rL LLVM

http://reviews.llvm.org/D11220







More information about the llvm-commits mailing list