[PATCH] D62731: Add support for options -frounding-math, -ftrapping-math, -ffp-model=, and -ffp-exception-behavior=, : Specify floating point behavior
Jordan Rupprecht via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Dec 12 12:51:37 PST 2019
rupprecht added inline comments.
================
Comment at: clang/include/clang/Basic/DiagnosticDriverKinds.td:444
+def warn_drv_experimental_fp_control_incomplete_opt : Warning<
+ "Support for floating point control option %0 is incomplete and experimental">,
----------------
mibintc wrote:
> @kpn thought it would be a good idea to add a Warning that the implementation of float control is experimental and partially implemented. That's what this is for.
Instead of adding a warning, I'd like to propose `-frounding-math` not be enabled unless an additional flag (e.g. `-fexperimental-float-control`) is passed. Or maybe this feature should be called `-f[no-]experimental-rounding-math` instead.
There are plenty of builds that are already specifying `-frounding-math` (e.g. they also support building w/ a compiler such as gcc that implements this), and adding this experimental/incomplete implementation is a surprise to those builds.
If I'm wrong and it's completely safe to ignore the warning (maybe it's incomplete but not in any unsafe way), we should just not have it at all.
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