This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, alpha]: Add TARGET_ATOMIC_ASSIGN_EXPAND_FENV hook
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: Richard Henderson <rth at redhat dot com>
- Date: Tue, 15 Jul 2014 18:04:26 +0200
- Subject: Re: [PATCH, alpha]: Add TARGET_ATOMIC_ASSIGN_EXPAND_FENV hook
- Authentication-results: sourceware.org; auth=none
- References: <CAFULd4ZYX2-qGYOjOFRCwyj4KPNkPqDvWuBznyfj3zDJorL-LQ at mail dot gmail dot com>
On Sat, Jul 12, 2014 at 12:19 AM, Uros Bizjak <ubizjak@gmail.com> wrote:
> Attached patch implements TARGET_ATOMIC_ASSIGN_EXPAND_FENV hook for
> alpha. The implementation is mostly copied from arm/arm.c, where calls
> to target builtins are replaced with calls to
> __ieee_{get,set}_fp_control function, as defined in OSF/1 ABI.
>
> The patch also includes a small fix to
> check_effective_target_fenv_exceptions, where we have to pass IEEE
> flags to successfully run the test function. Also, since the testcase
> gcc.dg/atomic/c11-atomic-exec-5.c takes a lot of time to finish with
> the default iteration count, the patch lowers the number of loops, so
> the testcase could finish in time (~30 seconds per invocation).
>
> 2014-07-11 Uros Bizjak <ubizjak@gmail.com>
>
> * config/alpha/alpha.c (alpha_atomic_assign_expand_fenv): New.
> (TARGET_ATOMIC_ASSIGN_EXPAND_FENV): New define.
>
> testsuite/ChangeLog:
>
> 2014-07-11 Uros Bizjak <ubizjak@gmail.com>
>
> * lib/target-supports.exp (check_effective_target_fenv_exceptions):
> Add IEEE options to compile flags.
> * gcc.dg/atomic/c11-atomic-exec-5.c: Ditto. Add -mieee-with-inexact
> additional option and lower ITER_COUNT to 100 for alpha*-*-* targets.
>
> Patch was bootstrapped and regression tested on alphaev68-pc-linux-gnu.
>
> OK for mainline and 4.9?
I have installed the patch on mainline under the assumption, that
maintainer has no objections.
Uros.