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

Guillaume Chatelet via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 3 01:32:20 PST 2020


gchatelet added inline comments.


================
Comment at: clang/docs/LanguageExtensions.rst:2252
+
+ * ``__builtin_memcpy_inline``
+
----------------
efriedma wrote:
> This is in the wrong section of the documentation.  We could constant-evaluate __builtin_memcpy_inline, I guess, but that isn't the primary purpose, and your patch doesn't implement constant evaluation anyway.
> 
> Might make sense to add a new section, if no existing section makes sense.
I added a new `memory builtins` section. Let me know what you think.


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