[PATCH] D76985: [LVI] Don't require DominatorTree in LVI (NFC)

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 18 01:34:03 PDT 2020


fhahn added inline comments.


================
Comment at: lib/Transforms/Scalar/JumpThreading.cpp:458
   // Flush only the Dominator Tree.
   DTU->getDomTree();
-  LVI->enableDT();
----------------
I think we flushed the DT updates here for LVI only? It would be good to drop this here (less frequent applying of updates should be quicker). If it is required somewhere else, we should move it there.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76985





More information about the llvm-commits mailing list