123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318 |
- //===-- VPlanVerifier.cpp -------------------------------------------------===//
- //
- // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
- // See https://llvm.org/LICENSE.txt for license information.
- // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
- //
- //===----------------------------------------------------------------------===//
- ///
- /// \file
- /// This file defines the class VPlanVerifier, which contains utility functions
- /// to check the consistency and invariants of a VPlan.
- ///
- //===----------------------------------------------------------------------===//
- #include "VPlanVerifier.h"
- #include "VPlan.h"
- #include "VPlanCFG.h"
- #include "llvm/ADT/DepthFirstIterator.h"
- #include "llvm/Support/CommandLine.h"
- #define DEBUG_TYPE "loop-vectorize"
- using namespace llvm;
- static cl::opt<bool> EnableHCFGVerifier("vplan-verify-hcfg", cl::init(false),
- cl::Hidden,
- cl::desc("Verify VPlan H-CFG."));
- #ifndef NDEBUG
- /// Utility function that checks whether \p VPBlockVec has duplicate
- /// VPBlockBases.
- static bool hasDuplicates(const SmallVectorImpl<VPBlockBase *> &VPBlockVec) {
- SmallDenseSet<const VPBlockBase *, 8> VPBlockSet;
- for (const auto *Block : VPBlockVec) {
- if (VPBlockSet.count(Block))
- return true;
- VPBlockSet.insert(Block);
- }
- return false;
- }
- #endif
- /// Helper function that verifies the CFG invariants of the VPBlockBases within
- /// \p Region. Checks in this function are generic for VPBlockBases. They are
- /// not specific for VPBasicBlocks or VPRegionBlocks.
- static void verifyBlocksInRegion(const VPRegionBlock *Region) {
- for (const VPBlockBase *VPB : vp_depth_first_shallow(Region->getEntry())) {
- // Check block's parent.
- assert(VPB->getParent() == Region && "VPBlockBase has wrong parent");
- auto *VPBB = dyn_cast<VPBasicBlock>(VPB);
- // Check block's condition bit.
- if (VPB->getNumSuccessors() > 1 || (VPBB && VPBB->isExiting()))
- assert(VPBB && VPBB->getTerminator() &&
- "Block has multiple successors but doesn't "
- "have a proper branch recipe!");
- else
- assert((!VPBB || !VPBB->getTerminator()) && "Unexpected branch recipe!");
- // Check block's successors.
- const auto &Successors = VPB->getSuccessors();
- // There must be only one instance of a successor in block's successor list.
- // TODO: This won't work for switch statements.
- assert(!hasDuplicates(Successors) &&
- "Multiple instances of the same successor.");
- for (const VPBlockBase *Succ : Successors) {
- // There must be a bi-directional link between block and successor.
- const auto &SuccPreds = Succ->getPredecessors();
- assert(llvm::is_contained(SuccPreds, VPB) && "Missing predecessor link.");
- (void)SuccPreds;
- }
- // Check block's predecessors.
- const auto &Predecessors = VPB->getPredecessors();
- // There must be only one instance of a predecessor in block's predecessor
- // list.
- // TODO: This won't work for switch statements.
- assert(!hasDuplicates(Predecessors) &&
- "Multiple instances of the same predecessor.");
- for (const VPBlockBase *Pred : Predecessors) {
- // Block and predecessor must be inside the same region.
- assert(Pred->getParent() == VPB->getParent() &&
- "Predecessor is not in the same region.");
- // There must be a bi-directional link between block and predecessor.
- const auto &PredSuccs = Pred->getSuccessors();
- assert(llvm::is_contained(PredSuccs, VPB) && "Missing successor link.");
- (void)PredSuccs;
- }
- }
- }
- /// Verify the CFG invariants of VPRegionBlock \p Region and its nested
- /// VPBlockBases. Do not recurse inside nested VPRegionBlocks.
- static void verifyRegion(const VPRegionBlock *Region) {
- const VPBlockBase *Entry = Region->getEntry();
- const VPBlockBase *Exiting = Region->getExiting();
- // Entry and Exiting shouldn't have any predecessor/successor, respectively.
- assert(!Entry->getNumPredecessors() && "Region entry has predecessors.");
- assert(!Exiting->getNumSuccessors() &&
- "Region exiting block has successors.");
- (void)Entry;
- (void)Exiting;
- verifyBlocksInRegion(Region);
- }
- /// Verify the CFG invariants of VPRegionBlock \p Region and its nested
- /// VPBlockBases. Recurse inside nested VPRegionBlocks.
- static void verifyRegionRec(const VPRegionBlock *Region) {
- verifyRegion(Region);
- // Recurse inside nested regions.
- for (const VPBlockBase *VPB : make_range(
- df_iterator<const VPBlockBase *>::begin(Region->getEntry()),
- df_iterator<const VPBlockBase *>::end(Region->getExiting()))) {
- if (const auto *SubRegion = dyn_cast<VPRegionBlock>(VPB))
- verifyRegionRec(SubRegion);
- }
- }
- void VPlanVerifier::verifyHierarchicalCFG(
- const VPRegionBlock *TopRegion) const {
- if (!EnableHCFGVerifier)
- return;
- LLVM_DEBUG(dbgs() << "Verifying VPlan H-CFG.\n");
- assert(!TopRegion->getParent() && "VPlan Top Region should have no parent.");
- verifyRegionRec(TopRegion);
- }
- // Verify that phi-like recipes are at the beginning of \p VPBB, with no
- // other recipes in between. Also check that only header blocks contain
- // VPHeaderPHIRecipes.
- static bool verifyPhiRecipes(const VPBasicBlock *VPBB) {
- auto RecipeI = VPBB->begin();
- auto End = VPBB->end();
- unsigned NumActiveLaneMaskPhiRecipes = 0;
- const VPRegionBlock *ParentR = VPBB->getParent();
- bool IsHeaderVPBB = ParentR && !ParentR->isReplicator() &&
- ParentR->getEntryBasicBlock() == VPBB;
- while (RecipeI != End && RecipeI->isPhi()) {
- if (isa<VPActiveLaneMaskPHIRecipe>(RecipeI))
- NumActiveLaneMaskPhiRecipes++;
- if (IsHeaderVPBB && !isa<VPHeaderPHIRecipe>(*RecipeI)) {
- errs() << "Found non-header PHI recipe in header VPBB";
- #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
- errs() << ": ";
- RecipeI->dump();
- #endif
- return false;
- }
- if (!IsHeaderVPBB && isa<VPHeaderPHIRecipe>(*RecipeI)) {
- errs() << "Found header PHI recipe in non-header VPBB";
- #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
- errs() << ": ";
- RecipeI->dump();
- #endif
- return false;
- }
- RecipeI++;
- }
- if (NumActiveLaneMaskPhiRecipes > 1) {
- errs() << "There should be no more than one VPActiveLaneMaskPHIRecipe";
- return false;
- }
- while (RecipeI != End) {
- if (RecipeI->isPhi() && !isa<VPBlendRecipe>(&*RecipeI)) {
- errs() << "Found phi-like recipe after non-phi recipe";
- #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
- errs() << ": ";
- RecipeI->dump();
- errs() << "after\n";
- std::prev(RecipeI)->dump();
- #endif
- return false;
- }
- RecipeI++;
- }
- return true;
- }
- static bool
- verifyVPBasicBlock(const VPBasicBlock *VPBB,
- DenseMap<const VPBlockBase *, unsigned> &BlockNumbering) {
- if (!verifyPhiRecipes(VPBB))
- return false;
- // Verify that defs in VPBB dominate all their uses. The current
- // implementation is still incomplete.
- DenseMap<const VPRecipeBase *, unsigned> RecipeNumbering;
- unsigned Cnt = 0;
- for (const VPRecipeBase &R : *VPBB)
- RecipeNumbering[&R] = Cnt++;
- for (const VPRecipeBase &R : *VPBB) {
- for (const VPValue *V : R.definedValues()) {
- for (const VPUser *U : V->users()) {
- auto *UI = dyn_cast<VPRecipeBase>(U);
- if (!UI || isa<VPHeaderPHIRecipe>(UI))
- continue;
- // If the user is in the same block, check it comes after R in the
- // block.
- if (UI->getParent() == VPBB) {
- if (RecipeNumbering[UI] < RecipeNumbering[&R]) {
- errs() << "Use before def!\n";
- return false;
- }
- continue;
- }
- // Skip blocks outside any region for now and blocks outside
- // replicate-regions.
- auto *ParentR = VPBB->getParent();
- if (!ParentR || !ParentR->isReplicator())
- continue;
- // For replicators, verify that VPPRedInstPHIRecipe defs are only used
- // in subsequent blocks.
- if (isa<VPPredInstPHIRecipe>(&R)) {
- auto I = BlockNumbering.find(UI->getParent());
- unsigned BlockNumber = I == BlockNumbering.end() ? std::numeric_limits<unsigned>::max() : I->second;
- if (BlockNumber < BlockNumbering[ParentR]) {
- errs() << "Use before def!\n";
- return false;
- }
- continue;
- }
- // All non-VPPredInstPHIRecipe recipes in the block must be used in
- // the replicate region only.
- if (UI->getParent()->getParent() != ParentR) {
- errs() << "Use before def!\n";
- return false;
- }
- }
- }
- }
- return true;
- }
- bool VPlanVerifier::verifyPlanIsValid(const VPlan &Plan) {
- DenseMap<const VPBlockBase *, unsigned> BlockNumbering;
- unsigned Cnt = 0;
- auto Iter = vp_depth_first_deep(Plan.getEntry());
- for (const VPBlockBase *VPB : Iter) {
- BlockNumbering[VPB] = Cnt++;
- auto *VPBB = dyn_cast<VPBasicBlock>(VPB);
- if (!VPBB)
- continue;
- if (!verifyVPBasicBlock(VPBB, BlockNumbering))
- return false;
- }
- const VPRegionBlock *TopRegion = Plan.getVectorLoopRegion();
- const VPBasicBlock *Entry = dyn_cast<VPBasicBlock>(TopRegion->getEntry());
- if (!Entry) {
- errs() << "VPlan entry block is not a VPBasicBlock\n";
- return false;
- }
- if (!isa<VPCanonicalIVPHIRecipe>(&*Entry->begin())) {
- errs() << "VPlan vector loop header does not start with a "
- "VPCanonicalIVPHIRecipe\n";
- return false;
- }
- const VPBasicBlock *Exiting = dyn_cast<VPBasicBlock>(TopRegion->getExiting());
- if (!Exiting) {
- errs() << "VPlan exiting block is not a VPBasicBlock\n";
- return false;
- }
- if (Exiting->empty()) {
- errs() << "VPlan vector loop exiting block must end with BranchOnCount or "
- "BranchOnCond VPInstruction but is empty\n";
- return false;
- }
- auto *LastInst = dyn_cast<VPInstruction>(std::prev(Exiting->end()));
- if (!LastInst || (LastInst->getOpcode() != VPInstruction::BranchOnCount &&
- LastInst->getOpcode() != VPInstruction::BranchOnCond)) {
- errs() << "VPlan vector loop exit must end with BranchOnCount or "
- "BranchOnCond VPInstruction\n";
- return false;
- }
- for (const VPRegionBlock *Region :
- VPBlockUtils::blocksOnly<const VPRegionBlock>(
- vp_depth_first_deep(Plan.getEntry()))) {
- if (Region->getEntry()->getNumPredecessors() != 0) {
- errs() << "region entry block has predecessors\n";
- return false;
- }
- if (Region->getExiting()->getNumSuccessors() != 0) {
- errs() << "region exiting block has successors\n";
- return false;
- }
- }
- for (const auto &KV : Plan.getLiveOuts())
- if (KV.second->getNumOperands() != 1) {
- errs() << "live outs must have a single operand\n";
- return false;
- }
- return true;
- }
|