Skip to content

Commit 0938622

Browse files
AndreyPavlenkoAndreyPavlenko
committed
Replace isF...() LLVM API calls with the corresponding isa<...>()
The isF...() methods have been removed in the main LLVM branch: llvm/llvm-project#123326
1 parent ec278de commit 0938622

File tree

15 files changed

+70
-58
lines changed

15 files changed

+70
-58
lines changed

‎include/triton/Conversion/MLIRTypes.h‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,15 @@ inline Type bf16Ty(MLIRContext *ctx) { return BFloat16Type::get(ctx); }
2828

2929
inline bool isFloat(Type type) {
3030
return type.isF32() || type.isF64() || type.isF16() || type.isF128() ||
31-
type.isBF16() || type.isFloat8E4M3B11FNUZ() || type.isFloat8E4M3FN() ||
32-
type.isFloat8E4M3FNUZ() || type.isFloat8E5M2() ||
33-
type.isFloat8E5M2FNUZ();
31+
type.isBF16() || isa<Float8E4M3B11FNUZType>(type) ||
32+
isa<Float8E4M3FNType>(type) || isa<Float8E4M3FNUZType>(type) ||
33+
isa<Float8E5M2Type>(type) || isa<Float8E5M2FNUZType>(type);
3434
}
3535

3636
inline bool isFloat8(Type type) {
37-
return type.isFloat8E4M3B11FNUZ() || type.isFloat8E4M3FN() ||
38-
type.isFloat8E4M3FNUZ() || type.isFloat8E5M2() ||
39-
type.isFloat8E5M2FNUZ();
37+
return isa<Float8E4M3B11FNUZType>(type) || isa<Float8E4M3FNType>(type) ||
38+
isa<Float8E4M3FNUZType>(type) || isa<Float8E5M2Type>(type) ||
39+
isa<Float8E5M2FNUZType>(type);
4040
}
4141

4242
inline bool isInt(Type type) { return type.isIntOrFloat() && !isFloat(type); }

‎lib/Analysis/Utility.cpp‎

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -738,14 +738,14 @@ bool supportMMA(triton::DotOp op, int version) {
738738
return false;
739739
if (!(numWarps % 4 == 0 && retShapePerCTA[rank - 2] % 64 == 0 &&
740740
retShapePerCTA[rank - 1] % 8 == 0 &&
741-
(aElemTy.isFloat8E5M2() || aElemTy.isFloat8E4M3FN() ||
741+
(isa<Float8E5M2Type>(aElemTy) || isa<Float8E4M3FNType>(aElemTy) ||
742742
aElemTy.isInteger(8) || aElemTy.isF16() || aElemTy.isBF16() ||
743743
aElemTy.isF32()))) {
744744
return false;
745745
}
746746
// We cannot use MMA_V3 if we need to accumulate in F32 within the MMA op.
747747
if (op.getMaxNumImpreciseAcc() < 32 &&
748-
(aElemTy.isFloat8E5M2() || aElemTy.isFloat8E4M3FN()) &&
748+
(isa<Float8E5M2Type>(aElemTy) || isa<Float8E4M3FNType>(aElemTy)) &&
749749
cast<RankedTensorType>(op.getType()).getElementType().isF32()) {
750750
return false;
751751
}
@@ -766,8 +766,9 @@ bool supportMMA(Value value, int version) {
766766
cast<triton::gpu::TensorOrMemDesc>(value.getType()).getElementType();
767767
// FP8 is not natively supported on all mma versions but it can always be
768768
// promoted to fp16 therefore we can always support it.
769-
bool isFP8 = elemTy.isFloat8E5M2() || elemTy.isFloat8E4M3FN() ||
770-
elemTy.isFloat8E5M2FNUZ() || elemTy.isFloat8E4M3FNUZ();
769+
bool isFP8 = isa<Float8E5M2Type>(elemTy) || isa<Float8E4M3FNType>(elemTy) ||
770+
isa<Float8E5M2FNUZType>(elemTy) ||
771+
isa<Float8E4M3FNUZType>(elemTy);
771772
return isFP8 || elemTy.isF16() || elemTy.isBF16() ||
772773
(elemTy.isF32() && version >= 2) ||
773774
(elemTy.isInteger(8) && version >= 2);

‎lib/Dialect/TritonGPU/Transforms/AccelerateMatmul.cpp‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -344,7 +344,8 @@ static void decomposeMixedModeDotOp(ModuleOp mod, int computeCapability) {
344344
NvidiaMmaEncodingAttr mmaLayout =
345345
dyn_cast<NvidiaMmaEncodingAttr>(D.getType().getEncoding());
346346
if (mmaLayout) {
347-
bool isNativeFP8 = AElType.isFloat8E5M2() || AElType.isFloat8E4M3FN();
347+
bool isNativeFP8 =
348+
isa<Float8E5M2Type>(AElType) || isa<Float8E4M3FNType>(AElType);
348349
// promote operands for sm < 89 since fp8 mma is not natively supported
349350
// promote operands for sm >= 90 when mma is not v3
350351
if (!isNativeFP8 ||

‎lib/Dialect/TritonGPU/Transforms/Utility.cpp‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,9 +44,9 @@ SmallVector<unsigned, 3> mmaVersionToInstrShape(int version,
4444
SmallVector<unsigned> validN;
4545

4646
// MMAv3 with larger instruction shape is preferred.
47-
if (eltType.isFloat8E5M2() || eltType.isFloat8E4M3FN() ||
48-
eltType.isFloat8E4M3FNUZ() || eltType.isF16() || eltType.isBF16() ||
49-
eltType.isF32()) {
47+
if (isa<Float8E5M2Type>(eltType) || isa<Float8E4M3FNType>(eltType) ||
48+
isa<Float8E4M3FNUZType>(eltType) || eltType.isF16() ||
49+
eltType.isBF16() || eltType.isF32()) {
5050
validN.assign({256, 248, 240, 232, 224, 216, 208, 200, 192, 184, 176,
5151
168, 160, 152, 144, 136, 128, 120, 112, 104, 96, 88,
5252
80, 72, 64, 56, 48, 40, 32, 24, 16, 8});

‎lib/Dialect/TritonNvidiaGPU/IR/Ops.cpp‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,8 @@ bool WarpGroupDotOp::needsPartialAccumulator() {
7777
const auto &d = getD();
7878
auto aTensorTy = cast<triton::gpu::TensorOrMemDesc>(a.getType());
7979
auto aElTy = cast<triton::gpu::TensorOrMemDesc>(a.getType()).getElementType();
80-
bool isFP8 = aElTy.isFloat8E5M2() || aElTy.isFloat8E4M3FN() ||
81-
aElTy.isFloat8E5M2FNUZ() || aElTy.isFloat8E4M3FNUZ();
80+
bool isFP8 = isa<Float8E5M2Type>(aElTy) || isa<Float8E4M3FNType>(aElTy) ||
81+
isa<Float8E5M2FNUZType>(aElTy) || isa<Float8E4M3FNUZType>(aElTy);
8282
bool accFP32 =
8383
cast<triton::gpu::TensorOrMemDesc>(d.getType()).getElementType().isF32();
8484
uint32_t maxNumImpreciseAcc = getMaxNumImpreciseAcc();

‎third_party/amd/lib/TritonAMDGPUToLLVM/ElementwiseOpToLLVM.cpp‎

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -985,17 +985,18 @@ struct FpToFpOpConversion
985985
return outVals;
986986
}
987987
size_t numElements = 4;
988-
if (srcElementType.isFloat8E4M3FN() || dstElementType.isFloat8E4M3FN() ||
989-
srcElementType.isFloat8E4M3FNUZ() ||
990-
dstElementType.isFloat8E4M3FNUZ() ||
991-
srcElementType.isFloat8E5M2FNUZ() ||
992-
dstElementType.isFloat8E5M2FNUZ()) {
988+
if (isa<Float8E4M3FNType>(srcElementType) ||
989+
isa<Float8E4M3FNType>(dstElementType) ||
990+
isa<Float8E4M3FNUZType>(srcElementType) ||
991+
isa<Float8E4M3FNUZType>(dstElementType) ||
992+
isa<Float8E5M2FNUZType>(srcElementType) ||
993+
isa<Float8E5M2FNUZType>(dstElementType)) {
993994
numElements = 2;
994995
}
995996
bool useFP16IntermediateSrc =
996997
srcElementType.isF32() && !(isaFamily == AMD::ISAFamily::CDNA3 &&
997-
(dstElementType.isFloat8E4M3FNUZ() ||
998-
dstElementType.isFloat8E5M2FNUZ()));
998+
(isa<Float8E4M3FNUZType>(dstElementType) ||
999+
isa<Float8E5M2FNUZType>(dstElementType)));
9991000
bool isDstFP32 = dstElementType.isF32();
10001001
Type srcType = useFP16IntermediateSrc ? f16_ty : srcElementType;
10011002
Type dstType = isDstFP32 ? f16_ty : dstElementType;

‎third_party/amd/lib/TritonAMDGPUTransforms/AccelerateAMDMatmul.cpp‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -416,7 +416,8 @@ class BlockedToMFMA : public OpRewritePattern<tt::DotOp> {
416416
// store instructions, except for fp8 matmul kernels due to regression
417417
// TODO (lixun): investigate the regression and enable this feature again
418418
auto aElemTy = mfmaInstr.getElementTypeA();
419-
bool isFP8 = aElemTy.isFloat8E5M2FNUZ() || aElemTy.isFloat8E4M3FNUZ();
419+
bool isFP8 =
420+
isa<Float8E5M2FNUZType>(aElemTy) || isa<Float8E4M3FNUZType>(aElemTy);
420421
bool isTransposed = isChainDot(dotOp) || !isFP8;
421422
mfmaEnc = ttg::AMDMfmaEncodingAttr::get(
422423
oldRetType.getContext(),

‎third_party/amd/lib/TritonAMDGPUTransforms/MfmaGroup.cpp‎

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,19 +20,23 @@ static MfmaTypeId chooseAppropriateMfmaId(mlir::Type dataTypeA,
2020
if (dataTypeA.isInteger(8) && dataTypeB.isInteger(8)) {
2121
return MfmaTypeId::I8TyId;
2222
}
23-
if (dataTypeA.isFloat8E4M3FNUZ() && dataTypeB.isFloat8E4M3FNUZ()) {
23+
if (isa<Float8E4M3FNUZType>(dataTypeA) &&
24+
isa<Float8E4M3FNUZType>(dataTypeB)) {
2425
return MfmaTypeId::Fp8Fp8TyId;
2526
}
26-
if (dataTypeA.isFloat8E4M3FNUZ() && dataTypeB.isFloat8E5M2FNUZ()) {
27+
if (isa<Float8E4M3FNUZType>(dataTypeA) &&
28+
isa<Float8E5M2FNUZType>(dataTypeB)) {
2729
return MfmaTypeId::Fp8Bf8TyId;
2830
}
29-
if (dataTypeA.isFloat8E5M2FNUZ() && dataTypeB.isFloat8E4M3FNUZ()) {
31+
if (isa<Float8E5M2FNUZType>(dataTypeA) &&
32+
isa<Float8E4M3FNUZType>(dataTypeB)) {
3033
return MfmaTypeId::Bf8Fp8TyId;
3134
}
32-
if (dataTypeA.isFloat8E5M2FNUZ() && dataTypeB.isFloat8E5M2FNUZ()) {
35+
if (isa<Float8E5M2FNUZType>(dataTypeA) &&
36+
isa<Float8E5M2FNUZType>(dataTypeB)) {
3337
return MfmaTypeId::Bf8Bf8TyId;
3438
}
35-
if (dataTypeA.isFloat8E5M2() && dataTypeB.isFloat8E5M2()) {
39+
if (isa<Float8E5M2Type>(dataTypeA) && isa<Float8E5M2Type>(dataTypeB)) {
3640
return MfmaTypeId::Fp16TyId;
3741
}
3842
llvm_unreachable("Unsupported input argument type.");

‎third_party/intel/lib/Analysis/DPAS.cpp‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -125,9 +125,9 @@ DPASAnalysis::getDPASType(OpTy op) {
125125
if (aElemTy.isF32() && op.getInputPrecision() == InputPrecision::TF32)
126126
return DPASEngineType::FP32_FP32_TF32_TF32;
127127
// For FP8XFP8->FP32, upcast to FP16
128-
if (aElemTy.isFloat8E5M2())
128+
if (isa<Float8E5M2Type>(aElemTy))
129129
return DPASEngineType::FP32_FP32_FP16_FP16;
130-
if (aElemTy.isFloat8E4M3FN())
130+
if (isa<Float8E4M3FNType>(aElemTy))
131131
return DPASEngineType::FP32_FP32_FP16_FP16;
132132
} else if (dElemTy.isF16()) {
133133
if (aElemTy.isF16())
@@ -148,35 +148,35 @@ DPASAnalysis::getDPASType(OpTy op) {
148148
if (isa<FloatType>(dElemTy)) {
149149
if (dElemTy.isF32()) {
150150
if (aElemTy.isBF16() &&
151-
(bElemTy.isFloat8E4M3FN() || bElemTy.isFloat8E5M2()))
151+
(isa<Float8E4M3FNType>(bElemTy) || isa<Float8E5M2Type>(bElemTy)))
152152
return DPASEngineType::FP32_FP32_BF16_FP8;
153153
// 2 E2M1 are packed into 1 int8
154154
if (aElemTy.isBF16() && bElemTy.isInteger(8))
155155
return DPASEngineType::FP32_FP32_BF16_FP4;
156-
if ((aElemTy.isFloat8E4M3FN() || aElemTy.isFloat8E5M2()) &&
156+
if ((isa<Float8E4M3FNType>(aElemTy) || isa<Float8E5M2Type>(aElemTy)) &&
157157
bElemTy.isBF16())
158158
return DPASEngineType::FP32_FP32_FP8_BF16;
159159
if (aElemTy.isF16() &&
160-
(bElemTy.isFloat8E4M3FN() || bElemTy.isFloat8E5M2()))
160+
(isa<Float8E4M3FNType>(bElemTy) || isa<Float8E5M2Type>(bElemTy)))
161161
return DPASEngineType::FP32_FP32_FP16_FP8;
162162
// 2 E2M1 are packed into 1 int8
163163
if (aElemTy.isF16() && bElemTy.isInteger(8))
164164
return DPASEngineType::FP32_FP32_FP16_FP4;
165-
if ((aElemTy.isFloat8E4M3FN() || aElemTy.isFloat8E5M2()) &&
165+
if ((isa<Float8E4M3FNType>(aElemTy) || isa<Float8E5M2Type>(aElemTy)) &&
166166
bElemTy.isF16())
167167
return DPASEngineType::FP32_FP32_FP8_FP16;
168-
if ((aElemTy.isFloat8E4M3FN() || aElemTy.isFloat8E5M2()) &&
169-
(bElemTy.isFloat8E4M3FN() || bElemTy.isFloat8E5M2()))
168+
if ((isa<Float8E4M3FNType>(aElemTy) || isa<Float8E5M2Type>(aElemTy)) &&
169+
(isa<Float8E4M3FNType>(bElemTy) || isa<Float8E5M2Type>(bElemTy)))
170170
return DPASEngineType::FP32_FP32_FP8_FP8;
171-
if ((aElemTy.isFloat8E4M3FN() || aElemTy.isFloat8E5M2()) &&
171+
if ((isa<Float8E4M3FNType>(aElemTy) || isa<Float8E5M2Type>(aElemTy)) &&
172172
bElemTy.isInteger(8))
173173
return DPASEngineType::FP32_FP32_FP8_FP4;
174174
if (aElemTy.isInteger(8) && bElemTy.isBF16())
175175
return DPASEngineType::FP32_FP32_FP4_BF16;
176176
if (aElemTy.isInteger(8) && bElemTy.isF16())
177177
return DPASEngineType::FP32_FP32_FP4_FP16;
178178
if (aElemTy.isInteger(8) &&
179-
(bElemTy.isFloat8E4M3FN() || bElemTy.isFloat8E5M2()))
179+
(isa<Float8E4M3FNType>(bElemTy) || isa<Float8E5M2Type>(bElemTy)))
180180
return DPASEngineType::FP32_FP32_FP4_FP8;
181181
}
182182
}

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -396,7 +396,8 @@ unsigned DpasEncodingAttr::getOpsPerChannel(Type elemType) {
396396
assert(elemType.isIntOrFloat() && "unsupported type for DpasEncodingAttr");
397397

398398
unsigned dpasElemBitWidths = elemType.getIntOrFloatBitWidth();
399-
if (elemType.isFloat8E5M2() || elemType.isFloat8E4M3FN())
399+
if (llvm::isa<Float8E5M2Type>(elemType) ||
400+
llvm::isa<Float8E4M3FNType>(elemType))
400401
dpasElemBitWidths *= 2; // We are upcasting FP8 to FP16.
401402

402403
return DPASCapability::opsChanBitWidths / dpasElemBitWidths;

0 commit comments

Comments
 (0)