[PATCH] D51525: [ADT] Add a new class OwnedOrBorrowed<T>

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 30 22:10:35 PDT 2018


dblaikie added a subscriber: zturner.
dblaikie added a comment.

Yep, I'll ask for the explanation - at least at Google there's been some
pushback on adding such a general purpose construct to the core library on
the premise that there's often/usually a better design. Equally I've
certainly seen some places it felt kind of right, for sure.


https://reviews.llvm.org/D51525





More information about the llvm-commits mailing list