[PATCH, ARM] Fix NEON vset_lane for D registers

2011-05-03 Thread Julian Brown
Hi, This patch fixes vset_lane intrinsic variants for D-register sized variables. A typo meant that the wrong lane would be set in many circumstances. Tested manually only. OK to apply? Thanks, Julian ChangeLog gcc/ * config/arm/neon.md (vec_setmode_internal): Fix misplaced

Re: [PATCH, ARM] Fix NEON vset_lane for D registers

2011-05-03 Thread Ramana Radhakrishnan
On 03/05/11 13:49, Julian Brown wrote: Hi, This patch fixes vset_lane intrinsic variants for D-register sized variables. A typo meant that the wrong lane would be set in many circumstances. Tested manually only. OK to apply? Ok - yes this looks almost obvious but please do a sanity

Re: [PATCH, ARM] Fix NEON vset_lane for D registers

2011-05-03 Thread Richard Earnshaw
On Tue, 2011-05-03 at 13:49 +0100, Julian Brown wrote: Hi, This patch fixes vset_lane intrinsic variants for D-register sized variables. A typo meant that the wrong lane would be set in many circumstances. Tested manually only. OK to apply? Thanks, Julian ChangeLog gcc/

Re: [PATCH, ARM] Fix NEON vset_lane for D registers

2011-05-03 Thread Joseph S. Myers
On Tue, 3 May 2011, Richard Earnshaw wrote: gcc/ * config/arm/neon.md (vec_setmode_internal): Fix misplaced parenthesis in D-register case. Presumably this is a silent 'wrong-code' bug. If so, what about released compilers? And what about an execution testcase that fails