[libcxx-commits] [PATCH] D110866: [libc++abi] Mark __cxa_new_handler with _LIBCPP_SAFE_STATIC

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Sep 30 11:18:03 PDT 2021


ldionne created this revision.
ldionne requested review of this revision.
Herald added a project: libc++abi.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++abi.
Herald added a reviewer: libc++abi.

For consistency with the other handlers, and because requiring constant
initialization whenever we can is a good thing.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D110866

Files:
  libcxxabi/src/cxa_default_handlers.cpp


Index: libcxxabi/src/cxa_default_handlers.cpp
===================================================================
--- libcxxabi/src/cxa_default_handlers.cpp
+++ libcxxabi/src/cxa_default_handlers.cpp
@@ -105,7 +105,8 @@
 _LIBCXXABI_DATA_VIS
 _LIBCPP_SAFE_STATIC std::unexpected_handler __cxa_unexpected_handler = default_unexpected_handler;
 
-std::new_handler __cxa_new_handler = 0;
+_LIBCXXABI_DATA_VIS
+_LIBCPP_SAFE_STATIC std::new_handler __cxa_new_handler = 0;
 
 namespace std
 {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D110866.376296.patch
Type: text/x-patch
Size: 486 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20210930/e95c76b2/attachment-0001.bin>


More information about the libcxx-commits mailing list