This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, ciklplus]: Use -ffloat-store for 32bit x86 in cilk-plus/AN/builtin_fn_{custom,mutating}.c
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Richard Biener <richard dot guenther at gmail dot com>
- Cc: Jeff Law <law at redhat dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 25 Nov 2014 10:38:34 +0100
- Subject: Re: [PATCH, ciklplus]: Use -ffloat-store for 32bit x86 in cilk-plus/AN/builtin_fn_{custom,mutating}.c
- Authentication-results: sourceware.org; auth=none
- References: <CAFULd4am0XoiRmDT6PzhMZGj9B9ve-zh75pyZsrz_nj4hzTrsw at mail dot gmail dot com> <5473A42C dot 6030807 at redhat dot com> <CAFiYyc2Wvuprb2zvb0UsBx5zB8yejG7Uey06DRH8zrZeKvi2tg at mail dot gmail dot com>
On Tue, Nov 25, 2014 at 10:23 AM, Richard Biener
<richard.guenther@gmail.com> wrote:
> On Mon, Nov 24, 2014 at 10:33 PM, Jeff Law <law@redhat.com> wrote:
>> On 11/22/14 11:50, Uros Bizjak wrote:
>>>
>>> Hello!
>>>
>>> These two tests fix PR target/63847 [1], where x87 excess precision
>>> causes testcase to fail. The problem was triggered by -fpic, please
>>> see the PR for analysis.
>>>
>>> The patch adds -ffloat-store for 32bit x86 target, a standard and well
>>> tested solution for this problem.
>>>
>>> 2014-11-22 Uros Bizjak <ubizjak@gmail.com>
>>>
>>> PR target/63847
>>> * c-c++-common/cilk-plus/AN/builtin_fn_custom.c: Add -ffloat-store
>>> for 32bit x86 targets.
>>> * c-c++-common/cilk-plus/AN/builtin_fn_mutating.c: Ditto.
>>
>> OK.
>
> Don't we have -fexcess-precision=standard for this now?
Oh ... indeed. I will update the patch to enable it for all x86 targets.
Thanks,
Uros.