test structured bindings

PiperOrigin-RevId: 725127187
openxla · Feb 10, 2025 · 75d667c · 75d667c
1 parent 2cef0d9
commit 75d667c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/shardy/dialect/sdy/transforms/propagation/basic_propagation.cc b/shardy/dialect/sdy/transforms/propagation/basic_propagation.cc
@@ -614,11 +614,7 @@ LogicalResult BasicPropagationPassImpl::propagate(
       conservativePropagation, shardingGroupMap);
   // We only need a single iteration (and another to confirm convergence), since
   // we make sure ops whose sharding changes are added back to the worklist.
-  GreedyRewriteConfig config;
-  config.useTopDownTraversal = true;
-  config.enableRegionSimplification = mlir::GreedySimplifyRegionLevel::Disabled;
-  config.fold = false;
-  config.cseConstants = false;
+  GreedyRewriteConfig config{.useTopDownTraversal = true};
   if (failed(applyPatternsGreedily(moduleOp, std::move(patterns), config))) {
     // We should always converge in 2 iterations, if we don't, something is
     // wrong.

diff --git a/shardy/dialect/sdy/transforms/propagation/debugging/source_sharding.cc b/shardy/dialect/sdy/transforms/propagation/debugging/source_sharding.cc
@@ -317,7 +317,7 @@ void saveShardingOriginsOnModule(
                            builder.getDictionaryAttr(entries));
     }
     TypeSwitch<Operation*, void>(owningOp)
-        .Case<func::FuncOp>([&, value = value](func::FuncOp funcOp) {
+        .Case<func::FuncOp>([&](func::FuncOp funcOp) {
           funcOp.setArgAttr(cast<BlockArgument>(value).getArgNumber(),
                             kShardingOriginsAttr,
                             builder.getDictionaryAttr(entries));