================ @@ -1748,11 +1754,27 @@ LogicalResult WsloopOp::verify() { if (!isWrapper()) return emitOpError() << "must be a loop wrapper"; + auto wrapper = + llvm::dyn_cast_if_present<LoopWrapperInterface>((*this)->getParentOp()); + bool isCompositeWrapper = wrapper && wrapper.isWrapper() && ---------------- skatrak wrote:
Nit: I think the name could be improved for clarity. Same comment for `SimdOp::verify()`. ```suggestion bool isCompositeChildLeaf = wrapper && wrapper.isWrapper() && ``` https://github.com/llvm/llvm-project/pull/102341 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits