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

[Sanjoy Das via Phabricator via llvm-commits]

sanjoy 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
----------------
davide wrote:
> Maybe `a nullable value handle` is enough?
I want to be clear about *when* the value is nulled out.


https://reviews.llvm.org/D32267