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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
