[PATCH] D95382: [VPlan] Use VPUser to manage CondBit

[Gil Rapaport via Phabricator via llvm-commits]

gilr accepted this revision.
gilr added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:410
+  /// which is the branch condition.
+  VPUser CondBitUser;
 
----------------
Nit: IIRC the condition bit is only used by the native path (LV uses the BranchOnMask recipe instead), but LV still sets the condition bit in the call to `setTwoSuccessors` in `insertTwoBlocksAfter`, which will now create a redundant user of the condition bit. Not sure it is/will be a problem, but we could pass nullptr there to avoid it (requires removing the assert in `setTwoSuccessors`).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95382