This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [committed][aarch64][gcc][patch] Fix -Wpedantic issue with testcase.
> >>
> >> So I am curious as to why this shows up in Christophe's test but not
> >> in any of your test runs or indeed the testruns with our scripts.
> >>
The test was changed before sending out, the individual entries moved to
a macro. I then verified the code generation and did not rerun it through
the testsuite. So It didn't get the -pedantic added.
> >> Ideally this should have been caught by the testing you do , just
> >> curious as to why -Wpedantic is getting caught in Christophe's bot
> >> but not our bot. Searching through gcc.target/aarch64 I see no use of
> >> -Wpedantic in any of the exp files. So is this coming from the top
> >> level in your testing Christophe ?
> >>
> >
> > In my gcc.log, the compilation line is:
> > /xgcc -B//gcc/ /gcc/testsuite/gcc.target/aarch64/struct_cpy.c
> > -fno-diagnostics-show-caret -fdiagnostics-color=never -ansi
> > -pedantic-errors -ffat-lto-objects -S -o struct_cpy.s
> >
> > I do not override it at the top level.
>
> I now see aarch64.exp: set DEFAULT_CFLAGS " -ansi -pedantic-errors"
>
> So looks like a botched test run then. My bad..
>
> Ramana
>
>
> >
> >> Ramana
> >> >
> >> > Thanks,
> >> > Tamar
> >> >
> >> > gcc/testsuite/
> >> > 2018-07-06 Tamar Christina <tamar.christina@arm.com>
> >> >
> >> > * gcc.target/aarch64/struct_cpy.c: Remove ;.
> >> >
> >> > --