address more review comments

Author: srcarroll

mlir/lib/Dialect/SCF/TransformOps/SCFTransformOps.cpp

@@ -440,10 +440,8 @@ transform::LoopFuseSiblingOp::apply(transform::TransformRewriter &rewriter,
            << "source handle (got " << llvm::range_size(sourceOps) << ")";
   }
 
-  LoopLikeOpInterface target =
-      dyn_cast<LoopLikeOpInterface>(*targetOps.begin());
-  LoopLikeOpInterface source =
-      dyn_cast<LoopLikeOpInterface>(*sourceOps.begin());
+  auto target = dyn_cast<LoopLikeOpInterface>(*targetOps.begin());
+  auto source = dyn_cast<LoopLikeOpInterface>(*sourceOps.begin());
   if (!target || !source)
     return emitSilenceableFailure(target->getLoc())
            << "target or source is not a loop op";

mlir/lib/Dialect/SCF/Utils/Utils.cpp

@@ -1220,6 +1220,7 @@ scf::ForOp mlir::fuseIndependentSiblingForLoops(scf::ForOp target,
 // interface methods.
 scf::ParallelOp mlir::fuseIndependentSiblingParallelLoops(
     scf::ParallelOp target, scf::ParallelOp source, RewriterBase &rewriter) {
+  OpBuilder::InsertionGuard guard(rewriter);
   Block *block1 = target.getBody();
   Block *block2 = source.getBody();
   auto term1 = cast<scf::ReduceOp>(block1->getTerminator());