aaron.ballman added inline comments.
================ Comment at: include/clang/Basic/Builtins.def:983 // POSIX string.h +LIBBUILTIN(memccpy, "v*v*vC*iz", "f", "string.h", ALL_GNU_LANGUAGES) LIBBUILTIN(stpcpy, "c*c*cC*", "f", "string.h", ALL_GNU_LANGUAGES) ---------------- xbolva00 wrote: > aaron.ballman wrote: > > Isn't `memccpy` supported by Visual Studio? What should happen there? > ”This POSIX function is deprecated. Use the ISO C++ conformant _memccpy > instead.“ Thanks for verifying it's still supported (deprecated != unsupported). We do support other Microsoft builtins, so should this one be added? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68377/new/ https://reviews.llvm.org/D68377 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
