[PATCH] D151834: Include math-errno with fast-math
Zahira Ammarguellat via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 25 12:39:12 PDT 2023
zahiraam marked 2 inline comments as done.
zahiraam added inline comments.
================
Comment at: clang/include/clang/Basic/LangOptions.h:857
setAllowApproxFuncOverride(!Value);
+ setMathErrnoOverride(Value);
if (Value)
----------------
aaron.ballman wrote:
> zahiraam wrote:
> > aaron.ballman wrote:
> > > Everything else does `!Value`; is it intentional that you're using `Value` instead?
> > Yes. I want to set the value of math-errno to the value of the pragma in the source.
> Hmm, okay. Making sure I understand the logic, please excuse me if this is a dumb question. :-) So the idea here is that if fp_precise is on, we have to honor math errno, and if fp_precise is off, we can ignore math errno?
Correct.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151834/new/
https://reviews.llvm.org/D151834
More information about the cfe-commits
mailing list