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

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 6 08:08:00 PDT 2019

Previous message: [PATCH] D65788: [ADT] Remove O(1) requirement to range passed to llvm::size
Next message: [PATCH] D65704: DirectoryWatcher::create: Adding better error handling.
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

vsk added a comment.

+1 to @dblaikie’s comment. I don’t want to encourage folks to write code the slow/wrong way by making that way ergonomic. D54686 <https://reviews.llvm.org/D54686> was an attempt to move away from that.


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

https://reviews.llvm.org/D65788

Previous message: [PATCH] D65788: [ADT] Remove O(1) requirement to range passed to llvm::size
Next message: [PATCH] D65704: DirectoryWatcher::create: Adding better error handling.
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list