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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
