[libcxx-commits] [PATCH] D145589: [libc++] Make std::allocator_arg and friends conforming in C++17

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

In D145589#4178836 <https://reviews.llvm.org/D145589#4178836>, @philnik wrote:

> In D145589#4178824 <https://reviews.llvm.org/D145589#4178824>, @Mordante wrote:
>
>> LGTM, but I really would like to use a helper macro over `#ifdef`s.
>
> Another option would be to just always mark them `inline`. Both clang and GCC support this as an extension.

Really? I wasn't aware of that. In that case +1 for that solution.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145589