[PATCH AArch64 2/2] Replace temporary inline assembler for vget_high
Thu Sep 4 13:21:00 GMT 2014
Ah, right you are. (Curiously I still can't find Richard's previous message in
for first patch, gcc/testsuite/ChangeLog:
* gcc.target/aarch64/vget_high_1.c: New test.
* gcc.target/aarch64/vget_low_1.c: Likewise.
for second patch, gcc/ChangeLog:
* config/aarch64/arm_neon.h (__GET_HIGH): New macro.
(vget_high_f32, vget_high_f64, vget_high_p8, vget_high_p16,
vget_high_s8, vget_high_s16, vget_high_s32, vget_high_s64,
vget_high_u8, vget_high_u16, vget_high_u32, vget_high_u64):
Remove temporary __asm__ and reimplement.
Marcus Shawcroft wrote:
> On 12 August 2014 11:12, Alan Lawrence <email@example.com> wrote:
>> This patch replaces the current inline assembler for the vget_high
>> intrinsics in arm_neon.h with a sequence of other calls, in a similar
>> fashion to vget_low. Unlike the assembler, these are all transparent to the
>> front-end, so should enable better optimization through the mid-end.
>> Tested check-gcc and check-g++ and aarch64-none-elf and aarch64_be-none-elf
>> (including new tests in previous patch!).
> I think we are still waiting on ChangeLogs for this and the related patch?
More information about the Gcc-patches