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]

cpplib: Minor _Pragma fixes. Varargs macro and _Pragma tests.


This patch fixes a couple of minor nits with _Pragma: we were using
the input line number rather than the output line number, and we
weren't properly saving state in the case that the _Pragma appears in
a directive.

Tests include Jamie Lokier's varargs macros, and _Pragma.  For those
into wacky tests, check out the _Pragma within a _Pragma test. <g>

Neil.

	* cpplib.c (run_directive): Use correct line number for output
	of _Pragma.  Remember any in-progress directive.

	* gcc.dg/cpp/vararg2.c, gcc.dg/cpp/_Pragm1.c: New tests.	

Index: gcc/cpplib.c
===================================================================
RCS file: /cvs/gcc/egcs/gcc/cpplib.c,v
retrieving revision 1.212
diff -u -p -r1.212 cpplib.c
--- cpplib.c	2000/10/31 23:34:59	1.212
+++ cpplib.c	2000/11/01 18:48:12
@@ -349,9 +349,11 @@ run_directive (pfile, dir_no, buf, count
      size_t count;
      const char *name;
 {
+  unsigned int output_line = pfile->lexer_pos.output_line;
+
   if (cpp_push_buffer (pfile, (const U_CHAR *) buf, count) != NULL)
     {
-      const struct directive *dir = &dtable[dir_no];
+      const struct directive *dir = &dtable[dir_no], *orig_dir;
 
       if (name)
 	CPP_BUFFER (pfile)->nominal_fname = name;
@@ -360,7 +362,10 @@ run_directive (pfile, dir_no, buf, count
 
       /* A kludge to avoid line markers for _Pragma.  */
       if (dir_no == T_PRAGMA)
-	pfile->lexer_pos.output_line = CPP_BUFFER (pfile)->prev->lineno;
+	pfile->lexer_pos.output_line = output_line;
+
+      /* Save any in-process directive; _Pragma can appear in one.  */
+      orig_dir = pfile->directive;
 
       /* For _Pragma, the text is passed through preprocessing stage 3
 	 only, i.e. no trigraphs, no escaped newline removal, and no
@@ -371,8 +376,8 @@ run_directive (pfile, dir_no, buf, count
       pfile->state.prevent_expansion++;
       (void) (*dir->handler) (pfile);
       pfile->state.prevent_expansion--;
-      pfile->directive = 0;
-      pfile->state.in_directive = 0;
+      pfile->directive = orig_dir;
+      pfile->state.in_directive = orig_dir != 0;
 
       skip_rest_of_line (pfile);
       if (pfile->buffer->cur != pfile->buffer->rlimit)
Index: gcc/testsuite/gcc.dg/cpp/_Pragma1.c
===================================================================
RCS file: _Pragma1.c
diff -N _Pragma1.c
--- /dev/null	Tue May  5 13:32:27 1998
+++ _Pragma1.c	Wed Nov  1 10:48:26 2000
@@ -0,0 +1,41 @@
+/* Copyright (C) 2000 Free Software Foundation, Inc.  */
+
+/* { dg-do preprocess } */
+
+/* Tests the _Pragma operator.  Contributed by Neil Booth 1 Nov 2000.  */
+
+/* Within the preprocessor, the easy pragma to test is "poison".  */
+
+#pragma GCC poison p1
+p1				/* { dg-error "poisoned" } */
+
+/* Standard use of _Pragma.  */
+_Pragma ("GCC poison p2")
+p2				/* { dg-error "poisoned" } */
+
+/* I can see no reason it cannot appear in a directive.  Check we
+   enter the conditional.  Putting the _Pragma at EOL also catches
+   nasties like not saving current lexer state properly.  Also test
+   that wide strings are OK.  */
+#if 1 _Pragma (L"GCC poison p3")
+p3				/* { dg-error "poisoned" } */
+#endif
+
+#define M1 _Pragma ("GCC poison p4")
+p4				/* No problem; not yet poisoned.  */
+#define M2(x) _Pragma (#x)
+
+/* Now test macro expansion with embedded _Pragmas.  */
+M1 p4				/* { dg-error "poisoned" } */
+M2 (GCC poison p5) p5		/* { dg-error "poisoned" } */
+
+/* Look, ma!  These things even nest.  */
+_Pragma ("_Pragma (\"GCC poison p6\") GCC poison p7")
+p6				/* { dg-error "poisoned" } */
+p7				/* { dg-error "poisoned" } */
+
+/* Check we ignore them in false conditionals.  */
+#if 0
+_Pragma ("GCC poison p8")
+#endif
+p8				/* No problem.  */
Index: gcc/testsuite/gcc.dg/cpp/vararg2.c
===================================================================
RCS file: vararg2.c
diff -N vararg2.c
--- /dev/null	Tue May  5 13:32:27 1998
+++ vararg2.c	Wed Nov  1 10:48:26 2000
@@ -0,0 +1,131 @@
+/* { dg-do preprocess } */
+/* { dg-options "-std=gnu99" } */
+
+/* Jamie's varargs macros from hell.  Not for the faint of heart.
+   Great tests that C99 and GNU varargs give identical results.
+   Adapted to the testsuite by Neil Booth, 1 Nov 2000.  */
+
+/* Permission is granted to use, copy, modify and distribute this file
+   freely for any purpose whatsoever.  This file is free software, and
+   there's no warranty.
+
+   -- Jamie Lokier <jamie.lokier@cern.ch>, 25/Sep/2000.  */
+
+#define dup3(x)            x,x,x
+
+/* Count elements in a list (0 to 10 max). */
+#define gnu_count(y...)   _gnu_count1 ( , ##y)
+#define _gnu_count1(y...) _gnu_count2 (y,10,9,8,7,6,5,4,3,2,1,0)
+#define _gnu_count2(_,x0,x1,x2,x3,x4,x5,x6,x7,x8,x9,n,ys...) n
+
+/* Tail of a list. */
+#define gnu_tail(y...)    _gnu_tail (y)
+#define _gnu_tail(x,y...) y
+
+/* Repeat N times. */
+#define gnu_repeat(n, x) gnu_tail (_gnu_repeat (n, x))
+#define _gnu_repeat(n, x) _gnu_repeat_##n (x)
+#define _gnu_repeat_0(x)
+#define _gnu_repeat_1(x) ,x
+#define _gnu_repeat_2(x) ,x,x
+#define _gnu_repeat_3(x) ,x,x,x
+#define _gnu_repeat_4(x) ,x,x,x,x
+#define _gnu_repeat_5(x) ,x,x,x,x,x
+
+#define _gnu_keep(xs...) xs
+#define _gnu_discard(xs...)
+#define _gnu_split_r(n,xs...) _gnu_split_rd (n,_gnu_keep,_gnu_discard xs)
+#define _gnu_split_d(n,xs...) _gnu_split_rd (n,_gnu_discard,_gnu_keep xs)
+#define _gnu_split_rd(n,xs...) _gnu_split_##n (xs)
+#define _gnu_split_0(a,b,xs...) a() b(xs)
+#define _gnu_split_1(a,b,x0,xs...) a(x0) b(xs)
+#define _gnu_split_2(a,b,x0,x1,xs...) a(x0,x1) b(xs)
+#define _gnu_split_3(a,b,x0,x1,x2,xs...) a(x0,x1,x2) b(xs)
+#define _gnu_split_4(a,b,x0,x1,x2,x3,xs...) a(x0,x1,x2,x3) b(xs)
+#define _gnu_split_5(a,b,x0,x1,x2,x3,x4,xs...) a(x0,x1,x2,x3,x4) b(xs)
+
+/* List manipulations.  Surprise: index zero is the rightmost element. */
+#define gnu_take(n, xs...) \
+  _gnu_split_d (_gnu_count1 ( , ## xs), _gnu_repeat (n, _gnu_error) , ## xs)
+#define gnu_drop(n, xs...) \
+  _gnu_split_d (n,,_gnu_split_r  (_gnu_count1 ( , ## xs), _gnu_repeat (n, _gnu_error) , ## xs))
+#define gnu_index(pos, xs...) gnu_take (1, gnu_drop (pos , ## xs))
+
+/* C99 __VA_ARGS__ versions */
+#define c99_count(...)    _c99_count1 ( , ##__VA_ARGS__)/* If only ## worked.*/
+#define _c99_count1(...)  _c99_count2 (__VA_ARGS__,10,9,8,7,6,5,4,3,2,1,0)
+#define _c99_count2(_,x0,x1,x2,x3,x4,x5,x6,x7,x8,x9,n,...) n
+
+#define c99_tail(...)     _c99_tail (__VA_ARGS__)
+#define _c99_tail(x,...)  __VA_ARGS__
+
+/* Repeat N times. */
+#define c99_repeat(n, x) c99_tail (_c99_repeat (n, x))
+#define _c99_repeat(n, x) _c99_repeat_##n (x)
+#define _c99_repeat_0(x)
+#define _c99_repeat_1(x) ,x
+#define _c99_repeat_2(x) ,x,x
+#define _c99_repeat_3(x) ,x,x,x
+#define _c99_repeat_4(x) ,x,x,x,x
+#define _c99_repeat_5(x) ,x,x,x,x,x
+
+#define _c99_keep(...)    __VA_ARGS__
+#define _c99_discard(...)
+#define _c99_split_r(n,...) _c99_split_rd(n,_c99_keep,_c99_discard __VA_ARGS__)
+#define _c99_split_d(n,...) _c99_split_rd(n,_c99_discard,_c99_keep __VA_ARGS__)
+#define _c99_split_rd(n,...) _c99_split_##n (__VA_ARGS__)
+#define _c99_split_0(a,b,...) a() b(__VA_ARGS__)
+#define _c99_split_1(a,b,x0,...) a(x0) b(__VA_ARGS__)
+#define _c99_split_2(a,b,x0,x1,...) a(x0,x1) b(__VA_ARGS__)
+#define _c99_split_3(a,b,x0,x1,x2,...) a(x0,x1,x2) b(__VA_ARGS__)
+#define _c99_split_4(a,b,x0,x1,x2,x3,...) a(x0,x1,x2,x3) b(__VA_ARGS__)
+#define _c99_split_5(a,b,x0,x1,x2,x3,x4,...) a(x0,x1,x2,x3,x4) b(__VA_ARGS__)
+
+/* List manipulations.  Surprise: index zero is the rightmost element. */
+#define c99_take(n, ...) \
+  _c99_split_d (_c99_count1 ( , ## __VA_ARGS__), _c99_repeat (n, _c99_error) , ## __VA_ARGS__)
+#define c99_drop(n, ...) \
+  _c99_split_d (n,,_c99_split_r  (_c99_count1 ( , ## __VA_ARGS__), _c99_repeat (n, _c99_error) , ## __VA_ARGS__))
+#define c99_index(pos, ...) c99_take (1, c99_drop (pos , ## __VA_ARGS__))
+
+/************** Expansions **************/
+
+/* Correct answers are 0, 0, 1, 2, 10.  */
+#if _gnu_count1 () != 0 || gnu_count () != 0 || gnu_count (A) != 1 \
+    || gnu_count (,) != 2 || gnu_count (A, B, C, D, E, F, G, H, I, J) != 10
+#error gnu_count
+#endif
+
+/* Correct answers are empty, 'x'.  */
+#if gnu_repeat (0, 'x') gnu_repeat (1, 'x') != 'x'
+#error gnu_repeat
+#endif
+
+/* Correct answers are "e", "b", "a", empty.  */
+#if gnu_index (0, 'a', 'b', 'c', 'd', 'e') != 'e' \
+ || gnu_index (3, 'a', 'b', 'c', 'd', 'e') != 'b' \
+ || gnu_index (4, 'a', 'b', 'c', 'd', 'e') != 'a' \
+    gnu_index (5, 'a', 'b', 'c', 'd', 'e')
+#error gnu_index
+#endif
+
+/************* C99 tests *************/
+
+/* The answers are 0, 0, 1, 2, 10 as for the non-C99 version.  */
+#if _c99_count1 () != 0 || c99_count () != 0 || c99_count (A) != 1 \
+    || c99_count (,) != 2 || c99_count (A, B, C, D, E, F, G, H, I, J) != 10
+#error c99_count
+#endif
+
+/* Correct answers are empty, 'x'.  */
+#if c99_repeat (0, 'x') c99_repeat (1, 'x') != 'x'
+#error c99_repeat
+#endif
+
+/* Correct answers are "e", "b", "a", empty.  */
+#if c99_index (0, 'a', 'b', 'c', 'd', 'e') != 'e' \
+ || c99_index (3, 'a', 'b', 'c', 'd', 'e') != 'b' \
+ || c99_index (4, 'a', 'b', 'c', 'd', 'e') != 'a' \
+    c99_index (5, 'a', 'b', 'c', 'd', 'e')
+#error gnu_index
+#endif

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