[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