aaron.ballman added reviewers: rsmith, joerg, rjmccall, andrew.w.kaylor,
aaron.ballman.
aaron.ballman added a comment.
Thanks for this! I'm adding some more reviewers to the list to help get this
reviewed.
The patch doesn't seem to apply cleanly, so precommit CI isn't running on it.
Can you rectify that? Also, the change needs test coverage of some kind.
================
Comment at: clang/lib/Frontend/InitPreprocessor.cpp :1022
+ Builder.defineMacro("__NO_MATH_ERRNO__");
+
if (LangOpts.FastMath || LangOpts.FiniteMathOnly)
----------------
Does GCC gate on `-ffast-math`? My testing suggests that `-fno-math-errno`
alone is what sets this macro in GCC.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116337/new/
https://reviews.llvm.org/D116337
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
