[PATCH] D48598: [ADT] drop_begin: use std namespace. NFC.

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 26 10:26:50 PDT 2018


dblaikie added a subscriber: aaron.ballman.
dblaikie added a comment.

I think we now have adl_begin/adl_end helpers in the llvm namespace that
should do the right thigh (which is that the call should be made like
"using std::begin; ...begin(X)..." The same way swap is meant to be called
(this allows a default implementation in the std namespace and allows for
types to implement their own custom implementation in their own namespace
to be found via ADL))


Repository:
  rL LLVM

https://reviews.llvm.org/D48598





More information about the llvm-commits mailing list