On Tue, 7 Jan 2020, Andre Vieira (lists) wrote:

> What about the previous patch fixing the existing testism by skipping for
> big-endian arm using existing target supports?

I thought this was an update containing both - so yes, that's OK as well.

Richard.

> On 07/01/2020 07:31, Richard Biener wrote:
> > On Fri, 13 Dec 2019, Andre Vieira (lists) wrote:
> > 
> >> Consequently whilst looking at the list I noticed these two were missing.
> >>
> >> This OK for trunk?
> > 
> > Yes.
> > 
> > Thanks,
> > Richard.
> > 
> >> gcc/ChangeLog:
> >> 2019-12-13  Andre Vieira  <andre.simoesdiasvie...@arm.com>
> >>
> >>          * doc/sourcebuild.texi (arm_little_endian, arm_nothumb):
> >>          Documented existing target checks.
> >>
> >> On 13/12/2019 13:05, Andre Vieira (lists) wrote:
> >>> Thanks for pointing it out Rainer. Thanks to that reminder I noticed there
> >>> is a different way to achieve this without adding that extra target check.
> >>>
> >>> This OK?
> >>>
> >>> gcc/testsuite/ChangeLog:
> >>>
> >>> 2019-12-12  Andre Vieira  <andre.simoesdiasvie...@arm.com>
> >>>
> >>>           * gcc.dg/vect/vect-epilogues.c: XFAIL for arm big endian.
> >>>
> >>> On 12/12/2019 17:41, Rainer Orth wrote:
> >>>> Hi Andre,
> >>>>
> >>>>> gcc/testsuite/ChangeLog:
> >>>>>
> >>>>> 2019-12-12  Andre Vieira  <andre.simoesdiasvie...@arm.com>
> >>>>>
> >>>>>           * gcc.dg/vect/vect-epilogues.c: XFAIL for arm big endian.
> >>>>>           * lib/target-supports.exp
> >>>>> (check_effective_target_arm_big_endian):
> >>>>>           New target selector.
> >>>>
> >>>> as always, this needs documenting in sourcebuild.texi.
> >>>>
> >>>>      Rainer
> >>>>
> >>
> > 
> 

-- 
Richard Biener <rguent...@suse.de>
SUSE Software Solutions Germany GmbH, Maxfeldstrasse 5, 90409 Nuernberg,
Germany; GF: Felix Imendörffer; HRB 36809 (AG Nuernberg)

Reply via email to