@@ -562,12 +562,10 @@ transform::ConvertLayoutOp::apply(transform::TransformRewriter &rewriter,
562562 transform::TransformResults &results,
563563 transform::TransformState &state) {
564564 auto targetValues = state.getPayloadValues (getTarget ());
565- if (!llvm::hasSingleElement (targetValues)) {
565+ if (!llvm::hasSingleElement (targetValues))
566566 return emitDefiniteFailure ()
567567 << " requires exactly one target value handle (got "
568568 << llvm::range_size (targetValues) << " )" ;
569- }
570-
571569 auto value = *targetValues.begin ();
572570
573571 xegpu::LayoutAttr layoutAttr = nullptr ;
@@ -579,17 +577,15 @@ transform::ConvertLayoutOp::apply(transform::TransformRewriter &rewriter,
579577
580578 // Get load op.
581579 auto maybeLoadOp = findProducerOfType<xegpu::LoadNdOp>(value);
582- if (!maybeLoadOp) {
580+ if (!maybeLoadOp)
583581 return emitSilenceableFailure (getLoc ()) << " Could not find load op." ;
584- }
585582 auto loadOp = *maybeLoadOp;
586583 // Get load op operand value layout
587584 auto producerLayoutAttr =
588585 xegpu::getDistributeLayoutAttr (loadOp.getOperand (0 ));
589- if (!producerLayoutAttr) {
586+ if (!producerLayoutAttr)
590587 return emitSilenceableFailure (getLoc ())
591588 << " Operand producer op does not have a layout attr." ;
592- }
593589
594590 if (producerLayoutAttr != layoutAttr) {
595591 rewriter.setInsertionPointAfter (loadOp.getOperation ());
0 commit comments