[libcxx-commits] [PATCH] D135813: [libc++][PMR] Add availability macros

Nikolas Klauser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed Oct 19 02:29:02 PDT 2022


philnik added inline comments.


================
Comment at: libcxx/include/__memory_resource/unsynchronized_pool_resource.h:74
 
   _LIBCPP_HIDE_FROM_ABI ~unsynchronized_pool_resource() override { release(); }
 
----------------
Mordante wrote:
> This function probably needs to be marked too since it calls release.
> 
> I usually use the attribute on the entire class/struct.
Thanks for the info! I didn't know you could mark a class as unavailable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135813



More information about the libcxx-commits mailing list