Fixing typo in #21142 that was causing failures on MSVC. (#21211)

benvanik · web-flow · commit 57d61720368c · 2025-06-26T12:45:13.000-07:00
This was not MSVC-specific, just that MSVC caught it. It was undefined
behavior on other compilers.
diff --git a/compiler/src/iree/compiler/Codegen/Dialect/GPU/Transforms/ExpandUndistributedInnerTiles.cpp b/compiler/src/iree/compiler/Codegen/Dialect/GPU/Transforms/ExpandUndistributedInnerTiles.cpp
@@ -125,7 +125,7 @@ struct ExpandInnerTileShapes final : OpRewritePattern<Codegen::InnerTiledOp> {
     SmallVector<tensor::ExpandShapeOp> maybeExpands(numOperands, nullptr);
     SmallVector<Value> newOperands(tiledOp.getOperands());
     SmallVector<Attribute> newPermutations;
-    if (*permutationsAttr) {
+    if (permutationsAttr) {
       newPermutations = llvm::to_vector(*permutationsAttr);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,7 @@ struct ExpandInnerTileShapes final : OpRewritePattern<Codegen::InnerTiledOp> {`
`125`	`125`	`SmallVector<tensor::ExpandShapeOp> maybeExpands(numOperands, nullptr);`
`126`	`126`	`SmallVector<Value> newOperands(tiledOp.getOperands());`
`127`	`127`	`SmallVector<Attribute> newPermutations;`
`128`		`- if (*permutationsAttr) {`
	`128`	`+ if (permutationsAttr) {`
`129`	`129`	`newPermutations = llvm::to_vector(*permutationsAttr);`
`130`	`130`	`}`
`131`	`131`