[PATCH] D71635: [clang] Rename -frounding-math to -fexperimental-rounding-math and add -frounding-math back as a gcc-compat arg.

Jordan Rupprecht via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 17 15:58:29 PST 2019


rupprecht created this revision.
rupprecht added reviewers: mibintc, chandlerc, echristo, rjmccall, kpn, erichkeane, rsmith, andrew.w.kaylor.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

D62731 <https://reviews.llvm.org/D62731> implements an incomplete/experimental version of rounding math, but since the flag doesn't have "experimental" in the name, the feedback that it is experimental is in the form of -Wexperimental-float-control.

Since this flag is already being accepted as a gcc compatability arg, put the current implementation being `-f[no-]experimental-rounding-math` instead, and allow `-frounding-math` as an ignored arg. Because "experimental" is now in the flag name, remove the warning -- it's now clear that the feature is incomplete/experimental when invoking it.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D71635

Files:
  clang/docs/UsersManual.rst
  clang/include/clang/Basic/DiagnosticDriverKinds.td
  clang/include/clang/Basic/DiagnosticGroups.td
  clang/include/clang/Driver/Options.td
  clang/lib/Driver/ToolChains/Clang.cpp
  clang/test/CodeGen/fpconstrained.c
  clang/test/CodeGen/fpconstrained.cpp
  clang/test/Driver/clang_f_opts.c
  clang/test/Driver/fp-model.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D71635.234410.patch
Type: text/x-patch
Size: 15465 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191217/b714ecfb/attachment-0001.bin>


More information about the cfe-commits mailing list