This revision was automatically updated to reflect the committed changes.
Closed by commit rL323239: [CUDA] CUDA has no device-side library builtins.
(authored by tra, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D42319?vs=130697&id=
jlebar accepted this revision.
jlebar added a comment.
This revision is now accepted and ready to land.
Got it, thanks for the explanation.
https://reviews.llvm.org/D42319
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org
tra added a comment.
In https://reviews.llvm.org/D42319#983377, @jlebar wrote:
> How does this affect e.g. calling memcpy()? There isn't a standard library
> implementation of this on nvptx, but we do want calls to memcpy() to be
> lowered to llvm.memcpy so that they can be optimized.
We imp
jlebar added a comment.
How does this affect e.g. calling memcpy()? There isn't a standard library
implementation of this on nvptx, but we do want calls to memcpy() to be lowered
to llvm.memcpy so that they can be optimized.
https://reviews.llvm.org/D42319
_
tra created this revision.
tra added a reviewer: jlebar.
Herald added a subscriber: sanjoy.
We should (almost) never consider a device-side declaration to match a
builtin. If we do, the un-inlined device-side functions provided by
CUDA headers that ship with clang may be ignored. We may end up em
