[PATCH] D30582: [Driver] Restructure handling of -ffast-math and similar options

John Brawn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 3 08:51:55 PST 2017


john.brawn created this revision.

The way -ffast-math and the various related options to tweak floating-point handling are handled is inflexible and rather confusing. This patch restructures things so that we go through the options adjusting our idea of what's enabled as we go, instead of trying to figure each individual thing out by working backwards from the end, as this makes the behaviour of each individual option more clear.

Doing it this way also means we get gcc-compatible behaviour for when the __FAST_MATH__ and __FINITE_MATH_ONLY__ macros are defined, as they should depend on the final set of features that are enabled and not just on -ffast-math and -ffinite-math-only specifically.


Repository:
  rL LLVM

https://reviews.llvm.org/D30582

Files:
  lib/Driver/Tools.cpp
  test/Driver/fast-math.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30582.90491.patch
Type: text/x-patch
Size: 16019 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170303/df2ff560/attachment.bin>


More information about the cfe-commits mailing list