[PATCH] D73404: Basis of dropping uses in llvm.assume.

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

I think this makes sense, especially since lifetime markers will soon join assume as droppable users. That said, we need to improve the `nullptr` replacement for assume. I made some suggestions here https://reviews.llvm.org/D73832#1902199



================
Comment at: llvm/include/llvm/IR/User.h:224
+  /// from happening.
+  bool isDroppable() const;
+
----------------
We should list what those uses are, later maybe define an enum to restrict the selection.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73404/new/

https://reviews.llvm.org/D73404