[libcxx-commits] [libcxx] [libc++][memory_resource] Applied `[[nodiscard]]` (PR #172134)

[Nikolas Klauser via libcxx-commits]

================
@@ -75,7 +77,7 @@ class _LIBCPP_AVAILABILITY_PMR _LIBCPP_EXPORTED_FROM_ABI synchronized_pool_resou
     return __unsync_.deallocate(__p, __bytes, __align);
   }
 
-  bool do_is_equal(const memory_resource& __other) const noexcept override; // key function
+  [[nodiscard]] bool do_is_equal(const memory_resource& __other) const noexcept override; // key function
----------------
philnik777 wrote:

I'm not convinced we want to annotate the `do_` functions. These should basically never be called anyways.

https://github.com/llvm/llvm-project/pull/172134