[PATCH] D62731: Add support for options -frounding-math, -ftrapping-math, -ffp-model=, and -ffp-exception-behavior=, : Specify floating point behavior

Kevin P. Neal via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 18 13:15:44 PST 2019


kpn added a comment.

In D62731#1750427 <https://reviews.llvm.org/D62731#1750427>, @mibintc wrote:

> In D62731#1750412 <https://reviews.llvm.org/D62731#1750412>, @kpn wrote:
>
> > Does anyone think a warning is appropriate because the new flags are exercising experimental, incomplete code in both clang and llvm? The warning would be removed when we believe the feature is complete and ready to use.
>
>
> @kpn Can you say more about "incomplete code in ... clang".  I don't know what's missing from clang.


See D70256 <https://reviews.llvm.org/D70256>. Calls to clang's math builtins that are llvm intrinsics need to be changed to be calls to the constrained intrinsics. I've been waiting to submit the patches because they weren't testable without these command line options.

There may be other issues as well. I'm not sure.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62731





More information about the cfe-commits mailing list