Re: [PATCH][AArch64] vrnd*_f64 patch for stage-1

2014-02-26 Thread Marcus Shawcroft
On 21 February 2014 12:44, Alex Velenko alex.vele...@arm.com wrote: This patch adds vrnd*_f64 aarch64 intrinsics. A testcase for those intrinsics is added. Run a complete LE and BE regression run with no regressions. Is patch OK for stage-1? gcc/ 2014-02-21 Alex Velenko

Re: [PATCH][AArch64] vrnd*_f64 patch for stage-1

2014-02-21 Thread Alex Velenko
On 13/02/14 17:43, Richard Henderson wrote: On 02/13/2014 03:17 AM, Alex Velenko wrote: +/* Sets rmode field of FPCR control register to + FPROUNDING_ZERO. */ Comment is wrong, or at least misleading. +void __inline __attribute__ ((__always_inline__)) +set_rounding_mode (uint32_t mode)

[PATCH][AArch64] vrnd*_f64 patch for stage-1

2014-02-13 Thread Alex Velenko
Hi, This patch adds vrnd*_f64 aarch64 intrinsics. A testcase for those intrinsics is added. Run a complete LE and BE regression run with no regressions. Is patch OK for stage-1? 2014-02-13 Alex Velenko alex.vele...@arm.com gcc/ * config/aarch64/aarch64-builtins.c

Re: [PATCH][AArch64] vrnd*_f64 patch for stage-1

2014-02-13 Thread Richard Henderson
On 02/13/2014 03:17 AM, Alex Velenko wrote: +/* Sets rmode field of FPCR control register to + FPROUNDING_ZERO. */ Comment is wrong, or at least misleading. +void __inline __attribute__ ((__always_inline__)) +set_rounding_mode (uint32_t mode) +{ + uint32_t r; + + /* Read current