[libc-commits] [PATCH] D111913: [libc] add stpcpy and stpncpy

[Michael Jones via Phabricator via libc-commits]

michaelrj added inline comments.


================
Comment at: libc/src/string/stpcpy.cpp:31
+  // break the sanitizers.
+  SANITIZER_MEMORY_INITIALIZED(result, size);
+
----------------
sivachandra wrote:
> Not sure if this is a valid argument for `mempcpy`. As in, we might want `mempcpy` to be instrumented really.
I was figuring that since `mempcpy` is effectively just a call to `memcpy` it should be treated the same, but I have no strong opinion one way or another.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111913