> -----Original Message----- > From: Christophe Lyon [mailto:christophe.l...@linaro.org] > Sent: 23 September 2017 18:52 > To: Tamar Christina > Cc: 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 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> 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 > >> checks that a target only supports aligned access. > >> > >> As such the results need to be inverted otherwise the test runs in > >> exactly the wrong circumstances. > >> > >> Committed as r253073 under the GCC obvious rule. > > > > Hi Tamar, > >
Hi Christoph, > > This is not as obvious as we might think. This patch is causing tcl > > syntax errors, such as: > > error executing dg-final: expected boolean value but got "!1" > > Ack, I didn't see this before, I only checked for tests using arm_vect_no_misalign and retested those and the few That were failing before. Sorry about that. > > I plan to send a fix b/o next week. > > > > The attached patch would apply after reverting yours. > I've applied it against r253072 (just before your patch) and the results are > visible at: > http://people.linaro.org/~christophe.lyon/cross-validation/gcc-test- > patches/253072-hw-misalign2/report-build-info.html > > Do they match your expectations? It looks like a few testcases need to be > adjusted, or new bugs are uncovered. > > If the patch OK with a suitable ChangeLog entry? That looks OK to me, about ~40 new tests correctly run now, and I think the new failures are just test-isms but I'll Look at them more closely today. Thanks, Tamar > > Thanks, > > Christophe > > > Christophe > > > >> > >> Thanks, > >> Tamar > >> > >> gcc/ChangeLog > >> 2017-09-21 Tamar Christina <tamar.christ...@arm.com> > >> > >> PR testsuite/78421 > >> * lib/target-supports.exp > (check_effective_target_vect_hw_misalign): > >> Invert arm check. > >> > >> --