This revision was automatically updated to reflect the committed changes.
Closed by commit rL317558: [docs] Add section 'Half-Precision Floating Point'
(authored by SjoerdMeijer).
Changed prior to commit:
https://reviews.llvm.org/D35295?vs=121265&id=121854#toc
Repository:
rL LLVM
https://re
simon_tatham accepted this revision.
simon_tatham added a comment.
This revision is now accepted and ready to land.
LGTM now, thanks! (But other reviewers may still want to comment.)
https://reviews.llvm.org/D35295
___
cfe-commits mailing list
cfe-c
SjoerdMeijer updated this revision to Diff 121265.
SjoerdMeijer added a comment.
Many thanks for the reviews and suggestions! Comments addressed.
https://reviews.llvm.org/D35295
Files:
docs/LanguageExtensions.rst
Index: docs/LanguageExtensions.rst
===
simon_tatham added inline comments.
Comment at: docs/LanguageExtensions.rst:462
+``__fp16`` is a ARM C-Language Extension (ACLE) and ``_Float16`` is defined by
+the C standards committee, which should help portability between entire
+architectures. Also, ``_Float16`` arithmetic a
SjoerdMeijer updated this revision to Diff 121003.
SjoerdMeijer added a comment.
Addressed the comments about portability, and added a note that Float16 is
available in both C and C++ mode.
https://reviews.llvm.org/D35295
Files:
docs/LanguageExtensions.rst
Index: docs/LanguageExtensions.rs
SjoerdMeijer added a comment.
Good points. I am thinking about how to write this down.
I am not yet sure that `_Float16` can reduce portability. I think the behaviour
will depend on FLT_EVAL_METHOD. I.e., if your architecture supports
half-precision instructions, you would like to set FLT_EVAL_M
simon_tatham added inline comments.
Comment at: docs/LanguageExtensions.rst:457
+that arithmetic on ``_Float16`` is performed in half-precision, thus it is not
+a storage-only format. It is recommended that portable code use the
+``_Float16`` type.
I think the us
SjoerdMeijer updated this revision to Diff 106167.
SjoerdMeijer added a comment.
Thanks for review! Feedback addressed.
https://reviews.llvm.org/D35295
Files:
docs/LanguageExtensions.rst
Index: docs/LanguageExtensions.rst
===
-
simon_tatham added inline comments.
Comment at: docs/LanguageExtensions.rst:448
+``__fp16`` is a storage and interchange format only. This means that values of
+``__fp16`` promote to (at least) float when used in artimethic operations.
There are
+two ``__fp16`` formats. Clang su
SjoerdMeijer created this revision.
This documents the differences and interactions between _Float16 and __fp16.
https://reviews.llvm.org/D35295
Files:
docs/LanguageExtensions.rst
Index: docs/LanguageExtensions.rst
===
--- docs
10 matches
Mail list logo