This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, mudflap] "Fix" libmudflap.c/pass46-frag.c FAILs
- From: Richard Guenther <richard dot guenther at gmail dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, "Frank Ch. Eigler" <fche at redhat dot com>
- Date: Sun, 16 May 2010 11:32:45 +0200
- Subject: Re: [PATCH, mudflap] "Fix" libmudflap.c/pass46-frag.c FAILs
- References: <4BEF0AB7.4040802@gmail.com> <4BEF0B62.4090105@gmail.com>
2010/5/15 Uros Bizjak <ubizjak@gmail.com>:
> [now included with the patch]
>
> Hello!
>
> This test uses -Wall, so we are just picking "pass46-frag.c:15:14:
> warning: array subscript is above array bounds [-Warray-bounds]"
>
> -Wall serves no purpose here (since Wmudflap is enabled by default), so
> proposed patch just removes warning flag.
>
> 2010-05-15 ?Uros Bizjak ?<ubizjak@gmail.com>
>
> ? ? ? ?* testsuite/libmudflap.c/pass46-frag.c (dg-options): Remove -Wall.
>
> Tested on x86_64-pc-linux-gnu. OK for mainline SVN and branches?
Ok. ...
> Uros.
>
> Index: testsuite/libmudflap.c/pass46-frag.c
> ===================================================================
> --- testsuite/libmudflap.c/pass46-frag.c ? ? ? ?(revision 159440)
> +++ testsuite/libmudflap.c/pass46-frag.c ? ? ? ?(working copy)
> @@ -15,4 +15,4 @@
> ?int m = boo [l % 2 + 12]; /* should not be instrumented */
> ?return m & strlen (""); /* a fancy way of saying "0" */
... that fancy way of saying 0 is surely folded and not fancy anymore
(if it ever was). So we'll happily DCE all of the function.
Richard.
> ?}
> -/* { dg-options "-fmudflap -fmudflapir -lmudflap -Wall" } */
> +/* { dg-options "-fmudflap -fmudflapir -lmudflap" } */
>
>
>