On 20/06/14 09:41, Marcus Shawcroft wrote: > On 16 June 2014 15:26, Kyrill Tkachov wrote: >> Hi all, >> >> I noticed that a few saturating math intrinsics in arm_neon.h for aarch64 >> have the wrong types, i.e. not what's mandated by the ACLE spec. >> >> This patch fixes that by adjusting the types of the builtin functions that >> those intrinsics map to (and in the process cleaning up the VCON iterator) >> and adding tests for the affected intrinsics. >> >> I realise it's quite big, but the changes are mostly uniform. >> >> Bootstrapped and tested aarch64-none-linux-gnu. >> >> Ok for trunk? > OK, can you prepare a 4.9 backport? > Cheers > /Marcus > Here it is. Now it applies cleanly to 4.9 and the tests are in gcc.target/aarch64 instead of gcc.target/aarch64/simd Tested on aarch64-none-elf and aarch64_be-none-elf. Ok to apply? Thanks, Kyrill