[PATCH] D95382: [VPlan] Make VPBlockBase a VPUser.

Gil Rapaport via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 7 13:44:01 PST 2021


gilr added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:381
 /// A VPBlockBase can be either a VPBasicBlock or a VPRegionBlock.
-class VPBlockBase {
+class VPBlockBase : public VPUser {
   friend class VPBlockUtils;
----------------
Would it suffice for VPBlockBase to have a VPUser rather than being one? (may also align more easily with terminators in the future)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95382



More information about the llvm-commits mailing list