On 13 January 2015 at 10:47, Richard Sandiford <richard.sandif...@arm.com> wrote: > Several sub-based patterns allowed the stack pointer to be the destination > but not the first source. This looked like an oversight; in all the patterns > changed here (but not for example in *sub_mul_imm_<mode>), the instruction > allows the stack pointer to appear in both positions. > > Tested on aarch64-linux-gnu. OK to install? > > Thanks, > Richard > > > gcc/ > * config/aarch64/aarch64.md (subsi3, *subsi3_uxtw, subdi3) > (*sub_<optab><ALLX:mode>_<GPI:mode>, *sub_<optab><SHORT:mode>_si_uxtw) > (*sub_<optab><ALLX:mode>_shft_<GPI:mode>) > (*sub_<optab><SHORT:mode>_shft_si_uxtw, *sub_<optab><mode>_multp2) > (*sub_<optab>si_multp2_uxtw, *sub_uxt<mode>_multp2) > (*sub_uxtsi_multp2_uxtw): Add stack pointer sources. > > gcc/testsuite/ > * gcc.target/aarch64/subsp.c: New test.
OK /Marcus
