[PATCH] D158093: [Sema] Clean up ActionResult type a little. NFCI

[Aaron Ballman via Phabricator via cfe-commits]

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