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

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman accepted this revision.
aaron.ballman added a comment.

LGTM aside from a few nits.



================
Comment at: clang/docs/UsersManual.rst:1411
 
+   Note: ``-ffast-math`` causes ``crtfastmath.o`` to be linked with code.
+
----------------



================
Comment at: clang/docs/UsersManual.rst:1460
+     ``-ffp-contract`` setting is determined by the default value of
+     ``-ffp-contract`.
+
----------------



================
Comment at: clang/docs/UsersManual.rst:1694
+``-ffast-math`` and ``-funsafe-math-optimizations`` cause ``crtfastmath.o`` to be
+linked,  which adds a static constructor that sets the FTZ/DAZ bits in MXCSR,
+affecting not only the current compilation unit but all static and shared
----------------



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

https://reviews.llvm.org/D123630