[libcxx-commits] [PATCH] D115995: [libcxx] Add deprecation notices to macros deprecated in P0883R2

Raul Tambre via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sat Dec 18 12:30:59 PST 2021


tambre added a comment.

Should I bump the version listed for P0883R2 to 14.0? This only adds an extra non-mandated deprecation warning.



================
Comment at: libcxx/include/atomic:2703
+#if _LIBCPP_STD_VER > 17 && !defined(_LIBCPP_DISABLE_DEPRECATION_WARNINGS)
+#pragma clang deprecated(ATOMIC_FLAG_INIT)
+#pragma clang deprecated(ATOMIC_VAR_INIT)
----------------
Separate macro deprecation macros didn't seem worth it as this is likely to be the only place we'd use them for a while. Since other compilers should simply ignore these guarding these shouldn't be necessary either.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115995



More information about the libcxx-commits mailing list