[PATCH] D101287: [SimplifyCFG] Expose sinking and hoisting helpers.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/include/llvm/Transforms/Utils/Local.h:186
+bool hoistThenElseCodeToIf(BranchInst *BI, const TargetTransformInfo &TTI,
+                           bool EqTermsOnly, DomTreeUpdater *DTU);
+
----------------
RKSimon wrote:
> Do we need DTU? 
Yes I think so. I think the function may have to update the DT, in case the terminators are hoisted (around llvm/lib/Transforms/Utils/SimplifyCFG.cpp:1617)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101287