[libcxx-commits] [PATCH] D98036: [libcxx] Apply pragma for silencing warning when building with clang-cl too

Martin Storsjö via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Mar 5 06:27:14 PST 2021

mstorsjo added a comment.

In D98036#2606546 <https://reviews.llvm.org/D98036#2606546>, @ldionne wrote:

> I assume this warning is necessary because some functions are defined but not used in different configurations? Like a function that would be always defined but only used when building for Windows or the opposite?

No, this is a header under src/filesystem, included by two cpp files in src/filesystem, that contains an anonymous namespace with helper functions (to avoid creating externally visible symbols) used by both src files - but not all of the functions are used by both src files. (And there's one function with multiple overloads, where one overload currently doesn't seem to be used at all, see `const path::value_type* unwrap(path::string_type const& s) { return s.c_str(); }` on line 111.)

  rG LLVM Github Monorepo



More information about the libcxx-commits mailing list