[PATCH] D84804: [SROA][Mem2Reg] Use efficient droppable use API (after D83976)

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 28 15:21:59 PDT 2020


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

The new removeIntrinsicUsers is much nicer; LGTM



================
Comment at: llvm/lib/Transforms/Scalar/SROA.cpp:3090
       // TODO For now we forget assumed information, this can be improved.
-      OldPtr->dropDroppableUsesByUser(II);
+      OldPtr->dropDroppableUsesIn(II);
       return true;
----------------
We could pass down the Use here, if we cared; the caller computes it. But this is okay for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84804



More information about the llvm-commits mailing list