[libcxx-commits] [PATCH] D95972: [libc++] Use init_priority(100) when possible

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Feb 8 07:56:33 PST 2021


ldionne added a subscriber: arthur.j.odwyer.
ldionne added inline comments.


================
Comment at: libcxx/src/experimental/memory_resource.cpp:79
 
-// When compiled in C++14 this initialization should be a constant expression.
-// Only in C++11 is "init_priority" needed to ensure initialization order.
-#if _LIBCPP_STD_VER > 11
-_LIBCPP_SAFE_STATIC
-#endif
-ResourceInitHelper res_init _LIBCPP_INIT_PRIORITY_MAX;
+_LIBCPP_SAFE_STATIC ResourceInitHelper res_init;
 
----------------
@arthur.j.odwyer Pinging you here since you wrote that code. Does this LGTY?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95972



More information about the libcxx-commits mailing list