This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 1/3] [ARC] Fix failing pr77309 for ARC700
- From: Claudiu Zissulescu <claziss at gmail dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Francois Bedard <fbedard at synopsys dot com>, "claziss at gmail dot com" <claziss at synopsys dot com>, Andrew Burgess <andrew dot burgess at embecosm dot com>
- Date: Thu, 21 Nov 2019 18:21:01 +0200
- Subject: Re: [PATCH 1/3] [ARC] Fix failing pr77309 for ARC700
- References: <20191119090250.5619-1-claziss@gmail.com> <ead58b8b-a8ed-da55-83e1-2eabbcae8972@redhat.com>
Thank you for the feedback.
Regarding the error, it is just spins when it tries to vectorize a
loop in tree-vect-slp.c. Quite strange ... I will look into asap. If
you have any hint please let me know :)
Claudiu
On Wed, Nov 20, 2019 at 7:41 PM Jeff Law <law@redhat.com> wrote:
>
> On 11/19/19 2:02 AM, Claudiu Zissulescu wrote:
> > The patterns neg_scc_insn and not_scc_insn are not correct, leading to
> > failing pr77309 test for ARC700. Add two new bic compare with zero
> > patterns to improve output code.
> >
> > gcc/
> > xxxx-xx-xx Claudiu Zissulescu <claziss@synopsys.com>
> >
> > * config/arc/arc.md (bic_f): Use cc_set_register predicate.
> > (bic_cmp0_noout): New pattern.
> > (bic_cmp0): Likewise.
> > (neg_scc_insn): Remove pattern.
> > (not_scc_insn): Likewise.
> OK.
>
> FYI between
>
> e00e0f8c391779c8d6cc9ad2fff8056a73c765c2 (good)
>
> fcae029b424f0546ee0efe574dff150be41271ea (bad)
>
> I got the following regression. I haven't looked into it yet at all,
> but figured passing it along might be helpful.
>
> > # Comparing 4 common sum files
> > ## /bin/sh gcc/contrib/compare_tests /tmp/gxx-sum1.60617 /tmp/gxx-sum2.60617
> > Tests that now fail, but worked before (1 tests):
> >
> > arc-sim: gcc.dg/pr86991.c (test for excess errors)
>
>
> Jeff
>