================
@@ -136,6 +136,7 @@ static constexpr fltSemantics semIEEEquad = {16383, -16382, 
113, 128};
 static constexpr fltSemantics semFloat8E5M2 = {15, -14, 3, 8};
 static constexpr fltSemantics semFloat8E5M2FNUZ = {
     15, -15, 3, 8, fltNonfiniteBehavior::NanOnly, 
fltNanEncoding::NegativeZero};
+static constexpr fltSemantics semFloat8E4M3 = {7, -6, 4, 8};
----------------
apivovarov wrote:

Maksim, Adding f8E4M3 IEE754 dtype was split into two PRs:

1. llvm/clang/APFloat changes 
[PR-97179](https://github.com/llvm/llvm-project/pull/97179) 
2. MLIR changes [PR-97118](https://github.com/llvm/llvm-project/pull/97118)  
(this PR)

PR-97118 depends on PR-97179. 

Currently you see two commits in PR-97118.

Once PR-97179 is merged - I'll rebase PR-97118 and it will contain just one 
commit - just MLIR changes.

Can you review just MLIR changes in this PR (PR-97118). All comments for 
llvm/clang/APFloat should go to 
[PR-97179](https://github.com/llvm/llvm-project/pull/97179).

https://github.com/llvm/llvm-project/pull/97118
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to