[PATCH] D102511: Do actual DCE in LoopUnroll

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/LoopUnroll.cpp:212
     SmallVector<WeakTrackingVH, 16> DeadInsts;
     simplifyLoopIVs(L, SE, DT, LI, TTI, DeadInsts);
 
----------------
Bikeshedding question: *why* do we do this here?
I'm looking at a test case where we spend most of the time (60%+, minutes) in LoopUnroll,
apparently in `simplifyLoopIVs()` specifically.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102511