This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch 5/11] Add -fpermitted-flt-eval-methods=[c11|ts-18661-3]
- From: Joseph Myers <joseph at codesourcery dot com>
- To: James Greenhalgh <james dot greenhalgh at arm dot com>
- Cc: <gcc-patches at gcc dot gnu dot org>, <nd at arm dot com>
- Date: Wed, 5 Oct 2016 22:59:44 +0000
- Subject: Re: [Patch 5/11] Add -fpermitted-flt-eval-methods=[c11|ts-18661-3]
- Authentication-results: sourceware.org; auth=none
- References: <1475254617-10825-1-git-send-email-james.greenhalgh@arm.com> <1475254617-10825-3-git-send-email-james.greenhalgh@arm.com>
On Fri, 30 Sep 2016, James Greenhalgh wrote:
> I've added two testcases which test that when this option, or a C standards
> dialect, would restrict the range of values to {-1, 0, 1, 2}, those are
> the only values we see. At this stage in the patch series this trivially
> holds for all targets.
As noted in my comments on patch 6, the tests should check FLT_EVAL_METHOD
from <float.h>, not the internal __FLT_EVAL_METHOD__ macro (and <float.h>
will need to contain conditionals so that the -std=c11
-D__STDC_WANT_IEC_60559_TYPES_EXT__ case uses the new values when
appropriate).
--
Joseph S. Myers
joseph@codesourcery.com