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. > + >
