Skip to content

Commit e00d0ab

Browse files
Fix build and test failures from 55cea61
Signed-off-by: Whitney Tsang <[email protected]>
1 parent 1af120b commit e00d0ab

File tree

4 files changed

+12
-12
lines changed

4 files changed

+12
-12
lines changed

test/Conversion/intel/tritongpu_to_gen.mlir

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -624,15 +624,15 @@ module attributes {"ttg.num-ctas" = 1 : i32, "ttg.num-warps" = 4 : i32} {
624624
// CHECK-NEXT: llvm.mlir.constant(1 : i32) : i32
625625
// CHECK-NEXT: llvm.mlir.constant(32 : i32) : i32
626626
// CHECK-NEXT: llvm.mlir.constant(512 : i32) : i32
627-
// CHECK-NEXT: llvm.add
628-
// CHECK-NEXT: llvm.add
629627
// CHECK-NEXT: llvm.mlir.constant(0 : i32) : i32
630628
// CHECK-NEXT: llvm.mul
631629
// CHECK-NEXT: llvm.add
632630
// CHECK-NEXT: llvm.mul
633631
// CHECK-NEXT: llvm.add
634632
// CHECK-NEXT: llvm.mul
635633
// CHECK-NEXT: llvm.add
634+
// CHECK-NEXT: llvm.add
635+
// CHECK-NEXT: llvm.add
636636
// CHECK-NEXT: llvm.getelementptr
637637
%index = arith.constant 1 : i32
638638
%zero = arith.constant 0 : i32

third_party/intel/lib/Analysis/Utility.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -175,12 +175,12 @@ bool cvtIsSubGroupShuffle(RankedTensorType srcTy, RankedTensorType dstTy) {
175175
StringAttr kBlock = str_attr("block");
176176

177177
std::optional<LinearLayout> srcLayout =
178-
toLinearLayout(srcTy.getShape(), srcTy.getEncoding());
178+
toLinearLayout(srcTy.getShape(), srcTy.getEncoding(), {});
179179
if (!srcLayout)
180180
return false;
181181

182182
std::optional<LinearLayout> dstLayout =
183-
toLinearLayout(dstTy.getShape(), dstTy.getEncoding());
183+
toLinearLayout(dstTy.getShape(), dstTy.getEncoding(), {});
184184
if (!dstLayout)
185185
return false;
186186

@@ -263,12 +263,12 @@ bool cvtIsSubGroupTranspose(RankedTensorType srcTy, RankedTensorType dstTy) {
263263
StringAttr kBlock = str_attr("block");
264264

265265
std::optional<LinearLayout> srcLayout =
266-
toLinearLayout(srcTy.getShape(), srcTy.getEncoding());
266+
toLinearLayout(srcTy.getShape(), srcTy.getEncoding(), {});
267267
if (!srcLayout)
268268
return false;
269269

270270
std::optional<LinearLayout> dstLayout =
271-
toLinearLayout(dstTy.getShape(), dstTy.getEncoding());
271+
toLinearLayout(dstTy.getShape(), dstTy.getEncoding(), {});
272272
if (!dstLayout)
273273
return false;
274274

third_party/intel/lib/Dialect/TritonIntelGPU/IR/Dialect.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1100,8 +1100,8 @@ struct TritonIntelGPUInferLayoutInterface
11001100
return success();
11011101
}
11021102
// Check whether the encodings are structurally the same.
1103-
const auto &expectedLL = triton::gpu::toLinearLayout(shape, expected);
1104-
const auto &gotLL = triton::gpu::toLinearLayout(shape, got);
1103+
const auto &expectedLL = triton::gpu::toLinearLayout(shape, expected, {});
1104+
const auto &gotLL = triton::gpu::toLinearLayout(shape, got, {});
11051105
if (expectedLL != gotLL) {
11061106
return emitOptionalError(loc, "Expected result encoding ", expected,
11071107
" but was ", got);
@@ -1123,7 +1123,7 @@ struct TritonIntelGPUInferLayoutInterface
11231123
// Once LinearLayouts are more widely used, we can remove
11241124
// inferReshapeOpLegacyEncoding and simply use LLs.
11251125
auto *ctx = getContext();
1126-
auto src = toLinearLayout(srcShape, srcEnc);
1126+
auto src = toLinearLayout(srcShape, srcEnc, {});
11271127

11281128
if (product(srcShape) != product(dstShape)) {
11291129
return emitOptionalError(loc, "numel of dst shape does not match "
@@ -1136,7 +1136,7 @@ struct TritonIntelGPUInferLayoutInterface
11361136
llvm::zip(standardOutDimNames(ctx, newRank), dstShape)) {
11371137
newOutDims.emplace_back(dim, size);
11381138
}
1139-
auto srcOutDims = to_vector(src.getOutDimNames());
1139+
auto srcOutDims = llvm::to_vector(src.getOutDimNames());
11401140
// reshapeOp assumes minor-to-major, so we need to transpose the out dims
11411141
// before the reshape
11421142
std::reverse(srcOutDims.begin(), srcOutDims.end());

third_party/intel/lib/TritonIntelGPUToLLVM/ConvertLayoutOpToLLVM.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,9 @@ struct ConvertLayoutOpUsingLinearLayoutsConversion
2929

3030
LinearLayout conversion = minimalCvtLayout(srcTy, dstTy);
3131
LinearLayout srcLayout =
32-
toLinearLayout(srcTy.getShape(), srcTy.getEncoding());
32+
toLinearLayout(srcTy.getShape(), srcTy.getEncoding(), {});
3333
LinearLayout dstLayout =
34-
toLinearLayout(dstTy.getShape(), dstTy.getEncoding());
34+
toLinearLayout(dstTy.getShape(), dstTy.getEncoding(), {});
3535

3636
StringAttr kLane = str_attr("lane");
3737

0 commit comments

Comments
 (0)