[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