//===- Verifier.cpp - MLIR Verifier Implementation ------------------------===// // // 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 // //===----------------------------------------------------------------------===// // // This file implements the verify() methods on the various IR types, performing // (potentially expensive) checks on the holistic structure of the code. This // can be used for detecting bugs in compiler transformations and hand written // .mlir files. // // The checks in this file are only for things that can occur as part of IR // transformations: e.g. violation of dominance information, malformed operation // attributes, etc. MLIR supports transformations moving IR through locally // invalid states (e.g. unlinking an operation from a block before re-inserting // it in a new place), but each transformation must complete with the IR in a // valid form. // // This should not check for things that are always wrong by construction (e.g. // attributes or other immutable structures that are incorrect), because those // are not mutable and can be checked at time of construction. // //===----------------------------------------------------------------------===// #include "mlir/IR/Verifier.h" #include "mlir/IR/Attributes.h" #include "mlir/IR/Dialect.h" #include "mlir/IR/Dominance.h" #include "mlir/IR/Operation.h" #include "mlir/IR/RegionKindInterface.h" #include "mlir/IR/Threading.h" #include "llvm/ADT/StringMap.h" #include "llvm/Support/FormatVariadic.h" #include "llvm/Support/PrettyStackTrace.h" #include "llvm/Support/Regex.h" #include #include using namespace mlir; namespace { /// This class encapsulates all the state used to verify an operation region. class OperationVerifier { public: /// If `verifyRecursively` is true, then this will also recursively verify /// nested operations. explicit OperationVerifier(bool verifyRecursively) : verifyRecursively(verifyRecursively) {} /// Verify the given operation. LogicalResult verifyOpAndDominance(Operation &op); private: /// Any ops that have regions and are marked as "isolated from above" will be /// returned in the opsWithIsolatedRegions vector. LogicalResult verifyBlock(Block &block, SmallVectorImpl &opsWithIsolatedRegions); /// Verify the properties and dominance relationships of this operation. LogicalResult verifyOperation(Operation &op); /// Verify the dominance property of regions contained within the given /// Operation. LogicalResult verifyDominanceOfContainedRegions(Operation &op, DominanceInfo &domInfo); /// A flag indicating if this verifier should recursively verify nested /// operations. bool verifyRecursively; }; } // namespace LogicalResult OperationVerifier::verifyOpAndDominance(Operation &op) { // Verify the operation first, collecting any IsolatedFromAbove operations. if (failed(verifyOperation(op))) return failure(); // Since everything looks structurally ok to this point, we do a dominance // check for any nested regions. We do this as a second pass since malformed // CFG's can cause dominator analysis construction to crash and we want the // verifier to be resilient to malformed code. if (op.getNumRegions() != 0) { DominanceInfo domInfo; if (failed(verifyDominanceOfContainedRegions(op, domInfo))) return failure(); } return success(); } /// Returns true if this block may be valid without terminator. That is if: /// - it does not have a parent region. /// - Or the parent region have a single block and: /// - This region does not have a parent op. /// - Or the parent op is unregistered. /// - Or the parent op has the NoTerminator trait. static bool mayBeValidWithoutTerminator(Block *block) { if (!block->getParent()) return true; if (!llvm::hasSingleElement(*block->getParent())) return false; Operation *op = block->getParentOp(); return !op || op->mightHaveTrait(); } LogicalResult OperationVerifier::verifyBlock( Block &block, SmallVectorImpl &opsWithIsolatedRegions) { for (auto arg : block.getArguments()) if (arg.getOwner() != &block) return emitError(arg.getLoc(), "block argument not owned by block"); // Verify that this block has a terminator. if (block.empty()) { if (mayBeValidWithoutTerminator(&block)) return success(); return emitError(block.getParent()->getLoc(), "empty block: expect at least a terminator"); } // Check each operation, and make sure there are no branches out of the // middle of this block. for (Operation &op : block) { // Only the last instructions is allowed to have successors. if (op.getNumSuccessors() != 0 && &op != &block.back()) return op.emitError( "operation with block successors must terminate its parent block"); // If we aren't verifying recursievly, there is nothing left to check. if (!verifyRecursively) continue; // If this operation has regions and is IsolatedFromAbove, we defer // checking. This allows us to parallelize verification better. if (op.getNumRegions() != 0 && op.hasTrait()) { opsWithIsolatedRegions.push_back(&op); // Otherwise, check the operation inline. } else if (failed(verifyOperation(op))) { return failure(); } } // Verify that this block is not branching to a block of a different // region. for (Block *successor : block.getSuccessors()) if (successor->getParent() != block.getParent()) return block.back().emitOpError( "branching to block of a different region"); // If this block doesn't have to have a terminator, don't require it. if (mayBeValidWithoutTerminator(&block)) return success(); Operation &terminator = block.back(); if (!terminator.mightHaveTrait()) return block.back().emitError("block with no terminator, has ") << terminator; return success(); } /// Verify the properties and dominance relationships of this operation, /// stopping region recursion at any "isolated from above operations". Any such /// ops are returned in the opsWithIsolatedRegions vector. LogicalResult OperationVerifier::verifyOperation(Operation &op) { // Check that operands are non-nil and structurally ok. for (auto operand : op.getOperands()) if (!operand) return op.emitError("null operand found"); /// Verify that all of the attributes are okay. for (auto attr : op.getDiscardableAttrDictionary()) { // Check for any optional dialect specific attributes. if (auto *dialect = attr.getNameDialect()) if (failed(dialect->verifyOperationAttribute(&op, attr))) return failure(); } // If we can get operation info for this, check the custom hook. OperationName opName = op.getName(); std::optional registeredInfo = opName.getRegisteredInfo(); if (registeredInfo && failed(registeredInfo->verifyInvariants(&op))) return failure(); SmallVector opsWithIsolatedRegions; if (unsigned numRegions = op.getNumRegions()) { auto kindInterface = dyn_cast(op); // Verify that all child regions are ok. MutableArrayRef regions = op.getRegions(); for (unsigned i = 0; i < numRegions; ++i) { Region ®ion = regions[i]; RegionKind kind = kindInterface ? kindInterface.getRegionKind(i) : RegionKind::SSACFG; // Check that Graph Regions only have a single basic block. This is // similar to the code in SingleBlockImplicitTerminator, but doesn't // require the trait to be specified. This arbitrary limitation is // designed to limit the number of cases that have to be handled by // transforms and conversions. if (op.isRegistered() && kind == RegionKind::Graph) { // Non-empty regions must contain a single basic block. if (!region.empty() && !region.hasOneBlock()) return op.emitOpError("expects graph region #") << i << " to have 0 or 1 blocks"; } if (region.empty()) continue; // Verify the first block has no predecessors. Block *firstBB = ®ion.front(); if (!firstBB->hasNoPredecessors()) return emitError(op.getLoc(), "entry block of region may not have predecessors"); // Verify each of the blocks within the region if we are verifying // recursively. if (verifyRecursively) { for (Block &block : region) if (failed(verifyBlock(block, opsWithIsolatedRegions))) return failure(); } } } // Verify the nested ops that are able to be verified in parallel. if (failed(failableParallelForEach( op.getContext(), opsWithIsolatedRegions, [&](Operation *op) { return verifyOpAndDominance(*op); }))) return failure(); // After the region ops are verified, run the verifiers that have additional // region invariants need to veirfy. if (registeredInfo && failed(registeredInfo->verifyRegionInvariants(&op))) return failure(); // If this is a registered operation, there is nothing left to do. if (registeredInfo) return success(); // Otherwise, verify that the parent dialect allows un-registered operations. Dialect *dialect = opName.getDialect(); if (!dialect) { if (!op.getContext()->allowsUnregisteredDialects()) { return op.emitOpError() << "created with unregistered dialect. If this is " "intended, please call allowUnregisteredDialects() on the " "MLIRContext, or use -allow-unregistered-dialect with " "the MLIR opt tool used"; } return success(); } if (!dialect->allowsUnknownOperations()) { return op.emitError("unregistered operation '") << op.getName() << "' found in dialect ('" << dialect->getNamespace() << "') that does not allow unknown operations"; } return success(); } //===----------------------------------------------------------------------===// // Dominance Checking //===----------------------------------------------------------------------===// /// Emit an error when the specified operand of the specified operation is an /// invalid use because of dominance properties. static void diagnoseInvalidOperandDominance(Operation &op, unsigned operandNo) { InFlightDiagnostic diag = op.emitError("operand #") << operandNo << " does not dominate this use"; Value operand = op.getOperand(operandNo); /// Attach a note to an in-flight diagnostic that provide more information /// about where an op operand is defined. if (auto *useOp = operand.getDefiningOp()) { Diagnostic ¬e = diag.attachNote(useOp->getLoc()); note << "operand defined here"; Block *block1 = op.getBlock(); Block *block2 = useOp->getBlock(); Region *region1 = block1->getParent(); Region *region2 = block2->getParent(); if (block1 == block2) note << " (op in the same block)"; else if (region1 == region2) note << " (op in the same region)"; else if (region2->isProperAncestor(region1)) note << " (op in a parent region)"; else if (region1->isProperAncestor(region2)) note << " (op in a child region)"; else note << " (op is neither in a parent nor in a child region)"; return; } // Block argument case. Block *block1 = op.getBlock(); Block *block2 = llvm::cast(operand).getOwner(); Region *region1 = block1->getParent(); Region *region2 = block2->getParent(); Location loc = UnknownLoc::get(op.getContext()); if (block2->getParentOp()) loc = block2->getParentOp()->getLoc(); Diagnostic ¬e = diag.attachNote(loc); if (!region2) { note << " (block without parent)"; return; } if (block1 == block2) llvm::report_fatal_error("Internal error in dominance verification"); int index = std::distance(region2->begin(), block2->getIterator()); note << "operand defined as a block argument (block #" << index; if (region1 == region2) note << " in the same region)"; else if (region2->isProperAncestor(region1)) note << " in a parent region)"; else if (region1->isProperAncestor(region2)) note << " in a child region)"; else note << " neither in a parent nor in a child region)"; } /// Verify the dominance of each of the nested blocks within the given operation LogicalResult OperationVerifier::verifyDominanceOfContainedRegions(Operation &op, DominanceInfo &domInfo) { for (Region ®ion : op.getRegions()) { // Verify the dominance of each of the held operations. for (Block &block : region) { // Dominance is only meaningful inside reachable blocks. bool isReachable = domInfo.isReachableFromEntry(&block); for (Operation &op : block) { if (isReachable) { // Check that operands properly dominate this use. for (const auto &operand : llvm::enumerate(op.getOperands())) { if (domInfo.properlyDominates(operand.value(), &op)) continue; diagnoseInvalidOperandDominance(op, operand.index()); return failure(); } } // Recursively verify dominance within each operation in the block, even // if the block itself is not reachable, or we are in a region which // doesn't respect dominance. if (verifyRecursively && op.getNumRegions() != 0) { // If this operation is IsolatedFromAbove, then we'll handle it in the // outer verification loop. if (op.hasTrait()) continue; if (failed(verifyDominanceOfContainedRegions(op, domInfo))) return failure(); } } } } return success(); } //===----------------------------------------------------------------------===// // Entrypoint //===----------------------------------------------------------------------===// LogicalResult mlir::verify(Operation *op, bool verifyRecursively) { OperationVerifier verifier(verifyRecursively); return verifier.verifyOpAndDominance(*op); }