>+ if (!TARGET_VFP) >+ return; >+ >+ /* Generate the equivalence of :
s/equivalence/equivalent. Ok with that change and if no regressions. Ramana On Mon, May 26, 2014 at 9:01 AM, Kugan <kugan.vivekanandara...@linaro.org> wrote: > Ping^2 ? > > Thanks, > Kugan > > On 12/05/14 09:47, Kugan wrote: >> Ping ? >> >> Thanks, >> Kugan >> >> On 02/05/14 19:04, Kugan wrote: >>> On 02/05/14 10:15, Joseph S. Myers wrote: >>>> It doesn't seem a good idea to me for a host-side GCC file to use the FE_* >>>> names for the target's FE_* values; you'd run into problems if that file >>>> ever ends up including the host's <fenv.h>, directly or indirectly, on any >>>> host. The same comment applies to the AArch64 patch as well. >>>> >>>> Instead I suggest names such as ARM_FE_* that won't conflict with the >>>> host's system headers. >>>> >>> Thanks for spotting it. Here is the updated patch that changes it to >>> ARM_FE_*. >>> >>> Thanks, >>> Kugan >>> >>> >>> gcc/ >>> >>> +2014-05-02 Kugan Vivekanandarajah <kug...@linaro.org> >>> + >>> + * config/arm/arm.c (TARGET_ATOMIC_ASSIGN_EXPAND_FENV): New define. >>> + (arm_builtins) : Add ARM_BUILTIN_GET_FPSCR and ARM_BUILTIN_SET_FPSCR. >>> + (bdesc_2arg) : Add description for builtins __builtins_arm_set_fpscr >>> + and __builtins_arm_get_fpscr. >>> + (arm_init_builtins) : Initialize builtins __builtins_arm_set_fpscr and >>> + __builtins_arm_get_fpscr. >>> + (arm_expand_builtin) : Expand builtins __builtins_arm_set_fpscr and >>> + __builtins_arm_ldfpscr. >>> + (arm_atomic_assign_expand_fenv): New function. >>> + * config/arm/vfp.md (set_fpscr): New pattern. >>> + (get_fpscr) : Likewise. >>> + * config/arm/unspecs.md (unspecv): Add VUNSPEC_GET_FPSCR and >>> + VUNSPEC_SET_FPSCR. >>> + * doc/extend.texi (AARCH64 Built-in Functions) : Document >>> + __builtins_arm_set_fpscr, __builtins_arm_get_fpscr. >>> + >>> >>
