[PATCH] D67517: Create UsersManual section entitled 'Controlling Floating Point Behavior'

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 13 07:38:23 PDT 2019


cameron.mcinally marked an inline comment as done.
cameron.mcinally added inline comments.


================
Comment at: clang/docs/UsersManual.rst:1152
+
+  This option implies:
+
----------------
mibintc wrote:
> rjmccall wrote:
> > To be clear, fast math is not *equivalent* to just setting all of these separately, right?
> I think in fact ffast-math is the same as selecting all the invidivdual settings, part of the logic checks if all the features are enabled and then also sets the fast-math macro. 
Just checked for crtfastmath.o on the link line and they do not seem to be "equivalent", unless I made a mistake.

That said, I think the language "implies" is acceptable here.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67517





More information about the llvm-commits mailing list