[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