aaron.ballman accepted this revision. aaron.ballman added a comment. Thank you for this! LGTM aside from a tiny nit.
================ Comment at: clang/include/clang/Sema/Ownership.h:147 +/// It may be: +/// - a valid pointer to the result object +/// - unset (null but valid), for constructs that may legitimately be absent ---------------- It might be worth tying this to `usable`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158093/new/ https://reviews.llvm.org/D158093 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
