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: [c/c++] PR 36513: add testcase


Hi!

You want to ask Janis... ;)

On Fri, Feb 19, 2010 at 12:38:11AM +0100, Manuel López-Ibáñez wrote:
> Done. OK to commit?
> 
> 2010-02-16  Manuel López-Ibáñez  <manu@gcc.gnu.org>
> 
> 	PR 36513
> testsuite/
> 	* c-c++-common/pr36513.c: New testcase.
> 	* c-c++-common/pr36513-2.c: New testcase.

> Index: gcc/testsuite/c-c++-common/pr36513-2.c
> ===================================================================
> --- gcc/testsuite/c-c++-common/pr36513-2.c	(revision 0)
> +++ gcc/testsuite/c-c++-common/pr36513-2.c	(revision 0)
> @@ -0,0 +1,13 @@
> +/* PR 36513: -Wlogical-op warns about strchr */
> +/* { dg-do compile } */
> +/* { dg-options "-Wlogical-op" } */
> +#ifdef __cplusplus
> +#include <cstring>
> +#else 
> +#include <string.h>
> +#endif
> +int main2 ()
> +{
> +  char *s, t;
> +  strchr (s, t);
> +}
> Index: gcc/testsuite/c-c++-common/pr36513.c
> ===================================================================
> --- gcc/testsuite/c-c++-common/pr36513.c	(revision 0)
> +++ gcc/testsuite/c-c++-common/pr36513.c	(revision 0)
> @@ -0,0 +1,15 @@
> +/* PR 36513: -Wlogical-op warns about strchr */
> +/* { dg-do compile } */
> +/* { dg-options "-Wlogical-op" } */
> +
> +extern void *__rawmemchr (const void *__s, int __c);
> +int main1 ()
> +{
> +  char *s, t;
> +  (__extension__ (__builtin_constant_p (t) 
> +		  && !__builtin_constant_p (s) 
> +		  && (t) == '\0' 
> +		  ? (char *) __rawmemchr (s, t) 
> +		  : __builtin_strchr (s, t)));
> +}
> +


	Jakub


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