[PATCH] D68377: [Builtins] Teach Clang about memccpy

Thu Oct 3 15:08:41 PDT 2019

xbolva00 marked an inline comment as done.
xbolva00 added inline comments.

================
Comment at: include/clang/Basic/Builtins.def:483
 BUILTIN(__builtin_memcpy, "v*v*vC*z", "nF")
+BUILTIN(__builtin_memccpy, "v*v*vC*iz", "nF")
 BUILTIN(__builtin_memmove, "v*v*vC*z", "nF")
----------------
aaron.ballman wrote:
> xbolva00 wrote:
> > aaron.ballman wrote:
> > > GCC doesn't seem to have `__builtin_memccpy`? https://godbolt.org/z/jbthQ3
> > Ok, I will drop it.
> If you drop it, won't that lose the builtin? I was mostly thinking it's in the wrong part of the list of builtins.
Rebuilding LLVM + Clang in progress so I just checked it in godbolt with "strtol" - defined only as LIBBUILTIN, no __builtin version.

nobuiltin attribute is correctly handled, so I think it will work. 
https://godbolt.org/z/Olfv-w

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

https://reviews.llvm.org/D68377