[PATCH] D70619: [NFC][LoopFusion] Use isControlFlowEquivalent() from CodeMoverUtils.

Whitney Tsang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 25 09:57:04 PST 2019


Whitney marked 2 inline comments as done.
Whitney added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/CodeMoverUtils.cpp:42
+                                   const PostDominatorTree &PDT) {
+  return ((DT.dominates(&BB0, &BB1) && PDT.dominates(&BB1, &BB0)) ||
+          (PDT.dominates(&BB0, &BB1) && DT.dominates(&BB1, &BB0)));
----------------
Meinersbur wrote:
> [suggestion] Add a shortcut:
> ```
>   if (&BB0 == &BB1)
>     return true;
> ```
Done the change when commit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70619





More information about the llvm-commits mailing list