[PATCH] D53909: ADT/STLExtras: Introduce llvm::empty(); NFC

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

(maybe some unit tests?)

Also, what about having only a single implementation:

  template <typename Range>
  constexpr bool empty(const Range &r) {
    return adl_begin(r) == adl_end(r);
  }

Not sure why the standard library doesn't do it this way, but there are probably good reasons... though maybe they don't apply to us/llvm?


Repository:
  rL LLVM

https://reviews.llvm.org/D53909