[PATCH] D73543: [clang] Add support for __builtin_memcpy_inline

Guillaume Chatelet via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 29 07:21:56 PST 2020


gchatelet added inline comments.


================
Comment at: clang/docs/LanguageExtensions.rst:2259
+``__builtin_memcpy_inline(dst, src, size)`` is identical to
+``__builtin_memcpy(dst, src, size)`` expect that the generated code is
+guaranteed not to call any external functions. See [LLVM IR ‘llvm.memcpy.inline’
----------------
arichardson wrote:
> Typo: except
Thx :)


================
Comment at: clang/test/Sema/builtins-memcpy-inline.c:23
+}
+
+void test_memcpy_inline_null_buffer_is_ok_if_size_is_zero(void *ptr) {
----------------
arichardson wrote:
> Size can only be a constant right? Should there be a test for the error reported in that case?
Thx it caught a bug indeed :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73543





More information about the cfe-commits mailing list