[llvm] [APFloat] Add APFloat support for E8M0 type (PR #107127)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 24 10:04:08 PDT 2024
================
@@ -1165,7 +1200,7 @@ IEEEFloat::IEEEFloat(const fltSemantics &ourSemantics, integerPart value) {
IEEEFloat::IEEEFloat(const fltSemantics &ourSemantics) {
initialize(&ourSemantics);
- makeZero(false);
+ ourSemantics.hasZero ? makeZero(false) : makeSmallestNormalized(false);
----------------
arsenm wrote:
It's not obvious to me that makeSmallestNormalized is the alternative. What's wrong with makeZero? You'll get a zero exponent, and an empty significand?
https://github.com/llvm/llvm-project/pull/107127
More information about the llvm-commits
mailing list