[PATCH] D32267: Add a new WeakVH value handle; NFC

[Davide Italiano via Phabricator via llvm-commits]

davide added inline comments.


================
Comment at: include/llvm/IR/ValueHandle.h:129-130
 
+/// \brief A nullable Value handle that is nullable.
+///
+/// This is a value handle that points to a value, and nulls itself
----------------
sanjoy wrote:
> davide wrote:
> > Maybe `a nullable value handle` is enough?
> I want to be clear about *when* the value is nulled out.
No objections.


https://reviews.llvm.org/D32267