[Mlir-commits] [mlir] [MLIR][AMDGPU] Add OCP FP8 support for new hardware (PR #106160)

Jakub Kuderski llvmlistbot at llvm.org
Thu Sep 19 10:51:22 PDT 2024


================
@@ -454,6 +454,20 @@ static void wmmaPushOutputOperand(ConversionPatternRewriter &rewriter,
   }
 }
 
+/// Return true if `type` is the E5M2 variant of an 8-bit float that is
+/// supported by the `_bf8` instructions on the given `chipset`.
+static bool isNativeBf8(Chipset chipset, Type type) {
----------------
kuhar wrote:

>  "is this type the same as the chipset's native BF8 type?"
Thanks for the explanation, I see the difference now. Maybe something like this then `isSupportedByNativeBf8(Chipset, Type)`?

If you prefer the current name, could you add a comment that explains what the intention is (similar to how you explained it above)?

https://github.com/llvm/llvm-project/pull/106160


More information about the Mlir-commits mailing list