This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch testsuite]: Adjust some tests of gcc.dg and gcc.c-torture for mingw targets
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Kai Tietz <ktietz70 at googlemail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 23 Feb 2012 18:06:02 +0100
- Subject: Re: [patch testsuite]: Adjust some tests of gcc.dg and gcc.c-torture for mingw targets
- References: <CAEwic4aCCkLOgfk0LMJuugBPubEuODx32EV7hjbiPgFQXLQDHw@mail.gmail.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Feb 23, 2012 at 05:56:11PM +0100, Kai Tietz wrote:
> --- gcc.dg/pack-test-5.c (revision 184486)
> +++ gcc.dg/pack-test-5.c (working copy)
> @@ -1,6 +1,7 @@
> /* PR c/11446: packed on a struct takes precedence over aligned on the type
> of a field. */
> /* { dg-do run } */
> +/* { dg-options "-mno-ms-bitfields" { target *-*-mingw* } } */
Shouldn't this be dg-additional-options instead?
> --- gcc.dg/Wpadded.c (revision 184486)
> +++ gcc.dg/Wpadded.c (working copy)
> @@ -6,6 +6,7 @@
> We won't get a warning anyway if the target has "packed" structure
> layout. */
> /* { dg-options "-Wpadded -fpack-struct=8" } */
> +/* { dg-options "-mno-ms-bitfields" { target *-*-mingw* } } */
And above too?
> --- gcc.dg/tls/opt-11.c (revision 184486)
> +++ gcc.dg/tls/opt-11.c (working copy)
> @@ -3,7 +3,7 @@
> /* { dg-add-options tls } */
>
> extern void abort (void);
> -extern void *memset (void *, int, __SIZE_TYPE__);
> +__extension__ extern void *memset (void *, int, __SIZE_TYPE__);
Why? I don't see extensions anywhere. Or is __SIZE_TYPE__ on mingw
long long and requires __extension__? If yes, it should at least go
right before the argument, but perhaps then it should be already
in the __SIZE_TYPE__ macro.
Jakub