[PATCH] D136176: Implement support for option 'fexcess-precision'.

[Zahira Ammarguellat via Phabricator via cfe-commits]

zahiraam added inline comments.


================
Comment at: clang/include/clang/Basic/LangOptions.h:300
+    FPP_Fast,
+    FPP_None
+  };
----------------
zahiraam wrote:
> andrew.w.kaylor wrote:
> > Is FPP_None somehow the same as fexcess-precision=16? If not, what does it mean?
> Yes, maybe this is confusing.  How about if we make "none" means disable excess precision and not use "16"? If you agree with this, then I will edit the description of the issue above.
> Is FPP_None somehow the same as fexcess-precision=16? If not, what does it mean?

I remember that @rjmccall wanted to have 16 a legal value so that we are compatible with GCC. Done that here.


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

https://reviews.llvm.org/D136176