[libcxx-commits] [PATCH] D71994: SFINAE span default constructor on Extent == 0

[Michael Schellenberger Costa via Phabricator via libcxx-commits]

miscco added a comment.

You are absolutely right.

While it is technically correct as both `span` and `concepts` are a C++20 feature I belive this change is not helpful. It is just soo much better :(

That said, we should switch to concepts once the library concepts and ranges are fully available as that is needed for conformance of the range based contructors.

How do I revert an update in phabricator?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71994