[libcxx-commits] [PATCH] D120634: [Libcxx] Add <source_location> header.

[James Y Knight via Phabricator via libcxx-commits]

jyknight marked 3 inline comments as done.
jyknight added a comment.

At this point, std::source_location::current() does still give the "wrong" result when used as a default argument.

After the discussion in https://reviews.llvm.org/D129488#3762490 about the unfortunate interaction between the specified behavior of source_location and consteval, the issue was brought to WG21 for comment, and is still under discussion there. I thought a possible outcome may be to switch `current()` from being consteval to instead being constexpr, but perhaps the consteval semantics will be changed, instead.

See also https://cplusplus.github.io/CWG/issues/2631.html


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120634