Re: [GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421)

2017-10-06 Thread Mike Stump
On Oct 6, 2017, at 5:48 AM, Tamar Christina wrote: > > I'm looking for permission to backport this patch to the gcc-7 branch to fix > the failing tests there as well. Ok.

Re: [GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421)

2017-10-06 Thread Tamar Christina
Tamar Christina; gcc-patches@gcc.gnu.org; nd; Ramana Radhakrishnan; Richard Earnshaw; ni...@redhat.com; Kyrylo Tkachov Subject: Re: [GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421) On Sep 25, 2017, at 9:58 PM, Christophe Lyon <christophe.l...@linaro.o

Re: [GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421)

2017-09-26 Thread Mike Stump
On Sep 25, 2017, at 9:58 PM, Christophe Lyon wrote: > > Yes, thanks! I was missing the 'expr' part. > > Here is what I have committed (r253187), to avoid further noise in the > results. Yup, looks good. Thanks.

Re: [GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421)

2017-09-25 Thread Christophe Lyon
On 25 September 2017 at 20:19, Mike Stump wrote: > On Sep 23, 2017, at 10:52 AM, Christophe Lyon > wrote: >> The attached patch would apply after reverting yours. >> I've applied it against r253072 (just before your patch) and the >> results

Re: [GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421)

2017-09-25 Thread Mike Stump
On Sep 23, 2017, at 10:52 AM, Christophe Lyon wrote: > The attached patch would apply after reverting yours. > I've applied it against r253072 (just before your patch) and the > results are visible at: >

Re: [GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421)

2017-09-25 Thread Christophe Lyon
gnu.org; nd; Ramana Radhakrishnan; Richard Earnshaw; >> ni...@redhat.com; Kyrylo Tkachov >> Subject: Re: [GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to >> misalign in vect_hw_misalign (PR 78421) >> >> On 23 September 2017 at 16:12, Christophe Lyon >>

RE: [GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421)

2017-09-25 Thread Tamar Christina
PATCH][TESTSUITE][ARM][COMMITTED] Invert check to > misalign in vect_hw_misalign (PR 78421) > > On 23 September 2017 at 16:12, Christophe Lyon > <christophe.l...@linaro.org> wrote: > > On 21 September 2017 at 16:48, Tamar Christina > <tamar.christ...@arm.com>

Re: [GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421)

2017-09-23 Thread Christophe Lyon
On 23 September 2017 at 16:12, Christophe Lyon wrote: > On 21 September 2017 at 16:48, Tamar Christina > wrote: >> Hi All, >> >> Commit r244796 changed vect_hw_misalign for arm to check against >> arm_vect_no_misalign. However

Re: [GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421)

2017-09-23 Thread Christophe Lyon
On 21 September 2017 at 16:48, Tamar Christina wrote: > Hi All, > > Commit r244796 changed vect_hw_misalign for arm to check against > arm_vect_no_misalign. However vect_hw_misalign is supposed to check if > a target supports misalign access, while arm_vect_no_misalign

[GCC][PATCH][TESTSUITE][ARM][COMMITTED] Invert check to misalign in vect_hw_misalign (PR 78421)

2017-09-21 Thread Tamar Christina
Hi All, Commit r244796 changed vect_hw_misalign for arm to check against arm_vect_no_misalign. However vect_hw_misalign is supposed to check if a target supports misalign access, while arm_vect_no_misalign checks that a target only supports aligned access. As such the results need to be inverted