[PATCH] D139927: [VPlan] Remove redundant blocks by merging them into predecessors.

Ayal Zaks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 26 14:02:27 PST 2022


Ayal accepted this revision.
Ayal added a comment.
This revision is now accepted and ready to land.

Thanks!



================
Comment at: llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:318
+      continue;
+    WorkList.push_back(VPBB);
+  }
----------------
nit: seems clearer w/o early-exit
```
    if (PredVPBB && PredVPBB->getNumSuccessors() == 1)
      WorkList.push_back(VPBB);
```
?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139927



More information about the llvm-commits mailing list