[PATCH] D65788: [ADT] Remove O(1) requirement to range passed to llvm::size

Alex Brachet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 6 07:59:01 PDT 2019


abrachet abandoned this revision.
abrachet added a comment.

> My preference would be to use std::distance(begin, end) for O(N) size computations to make it clear that they're "interesting", as I mentioned in the original review/discussion.

This makes a lot of sense to me.


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

https://reviews.llvm.org/D65788





More information about the llvm-commits mailing list