Re: [PATCH/AARCH64] Correctly handle stores of zero in fusion_load_store

2015-01-14 Thread Christophe Lyon
On 13 January 2015 at 13:29, Marcus Shawcroft wrote: > On 13 January 2015 at 04:48, Andrew Pinski wrote: > >> ChangeLog: >> * config/aarch64/aarch64.c (fusion_load_store): Check dest mode >> instead of src mode. >> >> >> * gcc.target/aarch64/store-pair-1.c: New testcase. > > OK, thanks /Marcus I

Re: [PATCH/AARCH64] Correctly handle stores of zero in fusion_load_store

2015-01-13 Thread Marcus Shawcroft
On 13 January 2015 at 04:48, Andrew Pinski wrote: > ChangeLog: > * config/aarch64/aarch64.c (fusion_load_store): Check dest mode > instead of src mode. > > > * gcc.target/aarch64/store-pair-1.c: New testcase. OK, thanks /Marcus

[PATCH/AARCH64] Correctly handle stores of zero in fusion_load_store

2015-01-12 Thread Andrew Pinski
Hi, With stores, we can get (const_int 0) as the src which has a mode of VOIDmode but this can be a valid store for optimizing into a store pair. This patch changes checking for the src mode to checking dest mode instead which does not have this issue. I added a testcase which shows the issue w