This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [committed][aarch64][gcc][patch] Fix -Wpedantic issue with testcase.


On Fri, Jul 6, 2018 at 10:16 AM, Tamar Christina
<tamar.christina@arm.com> wrote:
> Hi All,
>
> This fixes a -Wpedantic error with the testcase because of extra ; left after the
> functions.
>
> Regtested single test on aarch64-none-elf and no issues.
>
> Committed under the GCC obvious rule.


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.

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 ?

Ramana
>
> Thanks,
> Tamar
>
> gcc/testsuite/
> 2018-07-06  Tamar Christina  <tamar.christina@arm.com>
>
>         * gcc.target/aarch64/struct_cpy.c: Remove ;.
>
> --


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]