On Mon, Jun 2, 2014 at 5:47 PM, Charles Baylis <charles.bay...@linaro.org> wrote: > This patch adds support for post-indexed addressing for NEON structure > memory accesses. > > For example VLD1.8 {d0}, [r0], r1 > > > Bootstrapped and checked on arm-unknown-gnueabihf using Qemu. > > Ok for trunk?
This looks like a reasonable start but this work doesn't look complete to me yet. Can you also look at the impact on performance of a range of benchmarks especially a popular embedded one to see how this behaves unless you have already done so ? POST_INC, POST_MODIFY usually have a funny way of biting you with either ivopts or the way in which address costs work. I think there maybe further tweaks needed but for a first step I'd like to know what the performance impact is. I would also suggest running this through clyon's neon intrinsics testsuite to see if that catches any issues especially with the large vector modes. regards Ramana > > > gcc/Changelog: > > 2014-06-02 Charles Baylis <charles.bay...@linaro.org> > > * config/arm/arm.c (neon_vector_mem_operand): Allow register > POST_MODIFY for neon loads and stores. > (arm_print_operand): Output post-index register for neon loads and > stores.