================
@@ -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

Reply via email to