[libcxx-commits] [PATCH] D124011: [libc++] Always enable the ranges concepts

[Konstantin Varlamov via Phabricator via libcxx-commits]

var-const added a comment.

D118736 <https://reviews.llvm.org/D118736> deliberately made all the contents of the `ranges` namespace guarded by the macro, so I wouldn't undo that change lightly. It is a very valid argument that a user was broken by that patch. On the other hand, the main reason behind adding the macro was that we don't want users to start relying on ranges while the implementation is in progress and can (at least potentially) change in non-compatible ways. Let's wait for @ldionne to come back and discuss this patch then.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124011