[libc-commits] [PATCH] D74397: [libc] Adding memcpy implementation for x86_64

Guillaume Chatelet via Phabricator via libc-commits libc-commits at lists.llvm.org
Wed Feb 12 05:27:44 PST 2020


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


================
Comment at: libc/src/string/memcpy_arch_specific.h.def:17
+
+%%include_file(${memcpy_arch_specific})
+
----------------
@sivachandra  I'm not super happy with this pattern that dispatches the logic in many files. Would it be possible to generate the cpp file directly instead of generating an intermediate header file?

At least the common implementation would be in the template.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74397





More information about the libc-commits mailing list