@@ -430,12 +430,10 @@ transform::ConvertLayoutOp::apply(transform::TransformRewriter &rewriter,
430430 transform::TransformResults &results,
431431 transform::TransformState &state) {
432432 auto targetValues = state.getPayloadValues (getTarget ());
433- if (!llvm::hasSingleElement (targetValues)) {
433+ if (!llvm::hasSingleElement (targetValues))
434434 return emitDefiniteFailure ()
435435 << " requires exactly one target value handle (got "
436436 << llvm::range_size (targetValues) << " )" ;
437- }
438-
439437 auto value = *targetValues.begin ();
440438
441439 xegpu::LayoutAttr layoutAttr = nullptr ;
@@ -447,17 +445,15 @@ transform::ConvertLayoutOp::apply(transform::TransformRewriter &rewriter,
447445
448446 // Get load op.
449447 auto maybeLoadOp = findProducerOfType<xegpu::LoadNdOp>(value);
450- if (!maybeLoadOp) {
448+ if (!maybeLoadOp)
451449 return emitSilenceableFailure (getLoc ()) << " Could not find load op." ;
452- }
453450 auto loadOp = *maybeLoadOp;
454451 // Get load op operand value layout
455452 auto producerLayoutAttr =
456453 xegpu::getDistributeLayoutAttr (loadOp.getOperand (0 ));
457- if (!producerLayoutAttr) {
454+ if (!producerLayoutAttr)
458455 return emitSilenceableFailure (getLoc ())
459456 << " Operand producer op does not have a layout attr." ;
460- }
461457
462458 if (producerLayoutAttr != layoutAttr) {
463459 rewriter.setInsertionPointAfter (loadOp.getOperation ());
0 commit comments