[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