[libcxx-commits] [PATCH] D143797: [libc++] Mark some variables as inline, per the spec

[Louis Dionne via Phabricator via libcxx-commits]

ldionne marked an inline comment as done.
ldionne added inline comments.


================
Comment at: libcxx/include/__config:895
+#  if _LIBCPP_STD_VER >= 17
+#    define _LIBCPP_INLINE_VAR inline
+#  else
----------------
Mordante wrote:
> I wonder should we use this instead? And just `constexpr` in the `#else`. For me that makes it easier to understand what the macro does.
> 
The problem is that we still want those variables to be `constexpr` even in pre C++17.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143797