[libc-commits] [PATCH] D118954: [libc] add a vector internal class

[Tue Ly via Phabricator via libc-commits]

lntue added inline comments.


================
Comment at: libc/src/__support/CPP/CMakeLists.txt:14
+add_header_library(
+  cpp_with_malloc
+  HDRS
----------------
cpp_vector?


================
Comment at: libc/src/__support/CPP/vector.h:20
+
+void simple_memset(void *dst, char value, size_t count) {
+  char *data_array = static_cast<char *>(dst);
----------------
you'll need static inline for this to avoid multiple definitions / linking problems as it is defined in a header file.


================
Comment at: libc/src/__support/CPP/vector.h:20
+
+void simple_memset(void *dst, char value, size_t count) {
+  char *data_array = static_cast<char *>(dst);
----------------
lntue wrote:
> you'll need static inline for this to avoid multiple definitions / linking problems as it is defined in a header file.
Also is this being used anywhere?  It doesn't look like to be with vector class?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118954