[PATCH] D78190: Add Bfloat IR type

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 23 20:07:07 PDT 2020


rjmccall added a comment.

Thanks for doing it this way, I think this is going to be much better long-term for LLVM.



================
Comment at: llvm/docs/LangRef.rst:2896
+   * - ``bfloat``
+     - 16-bit brain floating-point value (8-bit mantissa)
+
----------------
scanon wrote:
> bfloat and fp128 should agree w.r.t. whether or not the implicit bit counts. Either 7 and 112 or 8 and 113. Also, we should use "significand" instead of "mantissa". "Mantissa" has slipped in in a bunch of places, but "significand" is the IEEE 754 terminology, and we should follow it.
I agree with Steve.  In general, there's no reason for these descriptions to be as terse as they are, especially for the non-standard formats.  Someone reading IR and seeing `bfloat` for the first time is going to come here and not be any wiser unless they figure out the right web search.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78190/new/

https://reviews.llvm.org/D78190





More information about the cfe-commits mailing list