[PATCH] D123630: Remove connection between 'ffast-math' and 'ffp-contract'.

[Joshua Cranmer via Phabricator via cfe-commits]

jcranmer-intel added inline comments.


================
Comment at: clang/docs/UsersManual.rst:1430
+
+   * ``-ffp-contract=on``
+
----------------
You can add `-fdenormal-fp-math=ieee` here.


================
Comment at: clang/docs/UsersManual.rst:1453-1455
+   Note: ``DenormalFPMath`` and ``DenormalFP32Math`` are set by default to IEEE
+   (no flush) for ``-fno-fast-math``, ``-fno-unsafe-math-optimizations``, and
+   any setting of ``fp-model``. Clang does enable flush-to-zero when
----------------
You can replace this text with saying that `-fno-fast-math` implies `-fdenormal-fp-math=ieee`. No need to directly mention `DenormalFPMath`; instead relate it to the other command line flags that are documented.


================
Comment at: clang/docs/UsersManual.rst:1455-1458
+   any setting of ``fp-model``. Clang does enable flush-to-zero when
+   ``-fast=math`` or ``-funsafe-math-optimzations`` are used, when it is able to
+   find the ``cftfastmath.o``. This will affect not only the current compilation
+   but all static and shared libraries included in the program.
----------------
`crtfastmath.o` should probably be mentioned in a separate section, like the "A note about ..." sections, with the text in `-fno-fast-math` only mentioning that it causes code not to be linked with `crtfastmath.o`.


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

https://reviews.llvm.org/D123630