]> git.saurik.com Git - bison.git/blame - TODO
TODO: statistics.
[bison.git] / TODO
CommitLineData
416bd7a9
MA
1-*- outline -*-
2
dd704c35 3* Short term
1e034807
AD
4** Use b4_symbol in all the skeleton
5Then remove the older system, including the tables generated by
6output.c
7
8** Update the documentation on gnu.org
9
10** Get rid of fake #lines [Bison: ...]
11Possibly as simple as checking whether the column number is nonnegative.
12
13I have seen messages like the following from GCC.
14
15<built-in>:0: fatal error: opening dependency file .deps/libltdl/argz.Tpo: No such file or directory
16
17
dd704c35
AD
18** Discuss about %printer/%destroy in the case of C++.
19It would be very nice to provide the symbol classes with an operator<<
20and a destructor. Unfortunately the syntax we have chosen for
21%destroy and %printer make them hard to reuse. For instance, the user
22is invited to write something like
23
24 %printer { debug_stream() << $$; } <my_type>;
25
26which is hard to reuse elsewhere since it wants to use
27"debug_stream()" to find the stream to use. The same applies to
28%destroy: we told the user she could use the members of the Parser
29class in the printers/destructors, which is not good for an operator<<
30since it is no longer bound to a particular parser, it's just a
31(standalone symbol).
32
33** Rename LR0.cc
34as lr0.cc, why upper case?
35
36** bench several bisons.
37Enhance bench.pl with %b to run different bisons.
38
39** Use b4_symbol everywhere.
40Move its definition in the more standard places and deploy it in other
41skeletons.
42
42f832d6 43* Various
865f1e9f
AD
44** YYPRINT
45glr.c inherits its symbol_print function from c.m4, which supports
46YYPRINT. But to use YYPRINT yytoknum is needed, which not defined by
47glr.c.
48
49Anyway, IMHO YYPRINT is obsolete and should be restricted to yacc.c.
50
42f832d6
AD
51** YYERRCODE
52Defined to 256, but not used, not documented. Probably the token
53number for the error token, which POSIX wants to be 256, but which
54Bison might renumber if the user used number 256. Keep fix and doc?
55Throw away?
56
90462b8d
AD
57Also, why don't we output the token name of the error token in the
58output? It is explicitly skipped:
59
60 /* Skip error token and tokens without identifier. */
61 if (sym != errtoken && id)
62
63Of course there are issues with name spaces, but if we disable we have
64something which seems to be more simpler and more consistent instead
65of the special case YYERRCODE.
66
67 enum yytokentype {
68 error = 256,
69 // ...
70 };
71
72
fc2476c7
AD
73We could (should?) also treat the case of the undef_token, which is
74numbered 257 for yylex, and 2 internal. Both appear for instance in
75toknum:
76
77 const unsigned short int
78 parser::yytoken_number_[] =
79 {
80 0, 256, 257, 258, 259, 260, 261, 262, 263, 264,
81
82while here
83
84 enum yytokentype {
85 TOK_EOF = 0,
86 TOK_EQ = 258,
87
88so both 256 and 257 are "mysterious".
89
90 const char*
91 const parser::yytname_[] =
92 {
93 "\"end of command\"", "error", "$undefined", "\"=\"", "\"break\"",
94
95
42f832d6
AD
96** YYFAIL
97It is seems to be *really* obsolete now, shall we remove it?
98
99** YYBACKUP
100There is no test about it, no examples in the doc, and I'm not sure
101what it should look like. For instance what follows crashes.
102
103 %error-verbose
104 %debug
105 %pure-parser
106 %code {
107 # include <stdio.h>
108 # include <stdlib.h>
109 # include <assert.h>
110
111 static void yyerror (const char *msg);
112 static int yylex (YYSTYPE *yylval);
113 }
114 %%
115 exp:
116 'a' { printf ("a: %d\n", $1); }
117 | 'b' { YYBACKUP('a', 123); }
118 ;
119 %%
120 static int
121 yylex (YYSTYPE *yylval)
122 {
123 static char const input[] = "b";
124 static size_t toknum;
125 assert (toknum < sizeof input);
126 *yylval = (toknum + 1) * 10;
127 return input[toknum++];
128 }
129
130 static void
131 yyerror (const char *msg)
132 {
133 fprintf (stderr, "%s\n", msg);
134 }
135
136 int
137 main (void)
138 {
139 yydebug = !!getenv("YYDEBUG");
140 return yyparse ();
141 }
142
27cb5b59
AD
143** yychar == yyempty_
144The code in yyerrlab reads:
145
146 if (yychar <= YYEOF)
147 {
148 /* Return failure if at end of input. */
149 if (yychar == YYEOF)
150 YYABORT;
151 }
152
153There are only two yychar that can be <= YYEOF: YYEMPTY and YYEOF.
154But I can't produce the situation where yychar is YYEMPTY here, is it
155really possible? The test suite does not exercise this case.
156
157This shows that it would be interesting to manage to install skeleton
158coverage analysis to the test suite.
42f832d6 159
a2e3fa77
AD
160** Table definitions
161It should be very easy to factor the definition of the various tables,
162including the separation bw declaration and definition. See for
163instance b4_table_define in lalr1.cc. This way, we could even factor
164C vs. C++ definitions.
165
00a8a083
AD
166* From lalr1.cc to yacc.c
167** Single stack
168Merging the three stacks in lalr1.cc simplified the code, prompted for
169other improvements and also made it faster (probably because memory
170management is performed once instead of three times). I suggest that
171we do the same in yacc.c.
172
dada3cd1 173** yysyntax_error
2b008529
AD
174The code bw glr.c and yacc.c is really alike, we can certainly factor
175some parts.
dada3cd1 176
3c146b5e
AD
177* Header guards
178
32f0598d 179From Franc,ois: should we keep the directory part in the CPP guard?
3c146b5e
AD
180
181
c19988b7
AD
182* Yacc.c: CPP Macros
183
184Do some people use YYPURE, YYLSP_NEEDED like we do in the test suite?
185They should not: it is not documented. But if they need to, let's
186find something clean (not like YYLSP_NEEDED...).
187
188
5d278082
PE
189* Installation
190
88bce5a2 191* Documentation
959e5f51
AD
192Before releasing, make sure the documentation ("Understanding your
193parser") refers to the current `output' format.
88bce5a2 194
2ab9a04f 195* Report
ec3bc396 196
c981ce9b
AD
197** Figures
198Some statistics about the grammar and the parser would be useful,
199especially when asking the user to send some information about the
200grammars she is working on. We should probably also include some
201information about the variables (I'm not sure for instance we even
202specify what LR variant was used).
203
2ab9a04f
AD
204** GLR
205How would Paul like to display the conflicted actions? In particular,
742e4900 206what when two reductions are possible on a given lookahead token, but one is
2ab9a04f
AD
207part of $default. Should we make the two reductions explicit, or just
208keep $default? See the following point.
d7215705 209
2ab9a04f
AD
210** Disabled Reductions
211See `tests/conflicts.at (Defaulted Conflicted Reduction)', and decide
212what we want to do.
d7215705 213
2ab9a04f 214** Documentation
bc933ef1
AD
215Extend with error productions. The hard part will probably be finding
216the right rule so that a single state does not exhibit too many yet
217undocumented ``features''. Maybe an empty action ought to be
218presented too. Shall we try to make a single grammar with all these
219features, or should we have several very small grammars?
ec3bc396 220
2ab9a04f
AD
221** --report=conflict-path
222Provide better assistance for understanding the conflicts by providing
223a sample text exhibiting the (LALR) ambiguity. See the paper from
224DeRemer and Penello: they already provide the algorithm.
225
38eb7751
PE
226** Statically check for potential ambiguities in GLR grammars. See
227<http://www.i3s.unice.fr/~schmitz/papers.html#expamb> for an approach.
228
ec3bc396 229
948be909 230* Extensions
2ab9a04f 231
d2aaf69e 232** Labeling the symbols
959e5f51
AD
233Have a look at the Lemon parser generator: instead of $1, $2 etc. they
234can name the values. This is much more pleasant. For instance:
235
236 exp (res): exp (a) '+' exp (b) { $res = $a + $b; };
237
238I love this. I have been bitten too often by the removal of the
239symbol, and forgetting to shift all the $n to $n-1. If you are
240unlucky, it compiles...
241
d2aaf69e
AD
242But instead of using $a etc., we can use regular variables. And
243instead of using (), I propose to use `:' (again). Paul suggests
244supporting `->' in addition to `:' to separate LHS and RHS. In other
245words:
246
247 r:exp -> a:exp '+' b:exp { r = a + b; };
248
249That requires an significant improvement of the grammar parser. Using
250GLR would be nice. It also requires that Bison know the type of the
251symbols (which will be useful for %include anyway). So we have some
252time before...
253
254Note that there remains the problem of locations: `@r'?
255
256
959e5f51
AD
257** $-1
258We should find a means to provide an access to values deep in the
259stack. For instance, instead of
260
261 baz: qux { $$ = $<foo>-1 + $<bar>0 + $1; }
262
263we should be able to have:
264
265 foo($foo) bar($bar) baz($bar): qux($qux) { $baz = $foo + $bar + $qux; }
266
267Or something like this.
268
f0e48240
AD
269** %if and the like
270It should be possible to have %if/%else/%endif. The implementation is
271not clear: should it be lexical or syntactic. Vadim Maslow thinks it
272must be in the scanner: we must not parse what is in a switched off
273part of %if. Akim Demaille thinks it should be in the parser, so as
274to avoid falling into another CPP mistake.
275
ca752c34
AD
276** XML Output
277There are couple of available extensions of Bison targeting some XML
278output. Some day we should consider including them. One issue is
279that they seem to be quite orthogonal to the parsing technique, and
280seem to depend mostly on the possibility to have some code triggered
281for each reduction. As a matter of fact, such hooks could also be
282used to generate the yydebug traces. Some generic scheme probably
283exists in there.
284
285XML output for GNU Bison and gcc
286 http://www.cs.may.ie/~jpower/Research/bisonXML/
287
288XML output for GNU Bison
289 http://yaxx.sourceforge.net/
f0e48240 290
fa770c86
AD
291* Unit rules
292Maybe we could expand unit rules, i.e., transform
293
294 exp: arith | bool;
295 arith: exp '+' exp;
296 bool: exp '&' exp;
297
298into
299
300 exp: exp '+' exp | exp '&' exp;
301
302when there are no actions. This can significantly speed up some
d7215705
AD
303grammars. I can't find the papers. In particular the book `LR
304parsing: Theory and Practice' is impossible to find, but according to
305`Parsing Techniques: a Practical Guide', it includes information about
306this issue. Does anybody have it?
fa770c86 307
51dec47b 308
51dec47b 309
2ab9a04f 310* Documentation
51dec47b 311
2ab9a04f
AD
312** History/Bibliography
313Some history of Bison and some bibliography would be most welcome.
314Are there any Texinfo standards for bibliography?
315
fa819509
AD
316** %printer
317Wow, %printer is not documented. Clearly mark YYPRINT as obsolete.
2ab9a04f 318
948be909
PE
319* Java, Fortran, etc.
320
948be909 321
2ab9a04f
AD
322* Coding system independence
323Paul notes:
324
325 Currently Bison assumes 8-bit bytes (i.e. that UCHAR_MAX is
326 255). It also assumes that the 8-bit character encoding is
327 the same for the invocation of 'bison' as it is for the
328 invocation of 'cc', but this is not necessarily true when
329 people run bison on an ASCII host and then use cc on an EBCDIC
330 host. I don't think these topics are worth our time
331 addressing (unless we find a gung-ho volunteer for EBCDIC or
332 PDP-10 ports :-) but they should probably be documented
333 somewhere.
fa770c86 334
d521d95a
PE
335 More importantly, Bison does not currently allow NUL bytes in
336 tokens, either via escapes (e.g., "x\0y") or via a NUL byte in
337 the source code. This should get fixed.
aef1ffd5 338
bcb05e75 339* --graph
45567173 340Show reductions.
bcb05e75 341
704a47c4 342* Broken options ?
45567173
AD
343** %token-table
344** Skeleton strategy
728c4be2 345Must we keep %token-table?
416bd7a9 346
0e95c1dd 347* BTYacc
f0e48240 348See if we can integrate backtracking in Bison. Charles-Henri de
df72984a
AD
349Boysson <de-boy_c@epita.fr> has been working on this, but never gave
350the results.
351
352Vadim Maslow, the maintainer of BTYacc was once contacted. Adjusting
353the Bison grammar parser will be needed to support some extra BTYacc
354features. This is less urgent.
0e95c1dd 355
2ab9a04f
AD
356** Keeping the conflicted actions
357First, analyze the differences between byacc and btyacc (I'm referring
358to the executables). Find where the conflicts are preserved.
359
360** Compare with the GLR tables
948be909 361See how isomorphic the way BTYacc and the way the GLR adjustments in
2ab9a04f
AD
362Bison are compatible. *As much as possible* one should try to use the
363same implementation in the Bison executables. I insist: it should be
364very feasible to use the very same conflict tables.
365
366** Adjust the skeletons
367Import the skeletons for C and C++.
368
0e95c1dd
AD
369
370* Precedence
2ab9a04f
AD
371
372** Partial order
0e95c1dd
AD
373It is unfortunate that there is a total order for precedence. It
374makes it impossible to have modular precedence information. We should
2ab9a04f 375move to partial orders (sounds like series/parallel orders to me).
0e95c1dd 376
2ab9a04f
AD
377** RR conflicts
378See if we can use precedence between rules to solve RR conflicts. See
379what POSIX says.
380
381
69991a58
AD
382* $undefined
383From Hans:
384- If the Bison generated parser experiences an undefined number in the
385character range, that character is written out in diagnostic messages, an
386addition to the $undefined value.
387
388Suggest: Change the name $undefined to undefined; looks better in outputs.
389
2ab9a04f 390
69991a58
AD
391* Default Action
392From Hans:
393- For use with my C++ parser, I transported the "switch (yyn)" statement
394that Bison writes to the bison.simple skeleton file. This way, I can remove
395the current default rule $$ = $1 implementation, which causes a double
396assignment to $$ which may not be OK under C++, replacing it with a
397"default:" part within the switch statement.
398
399Note that the default rule $$ = $1, when typed, is perfectly OK under C,
400but in the C++ implementation I made, this rule is different from
401$<type_name>$ = $<type_name>1. I therefore think that one should implement
402a Bison option where every typed default rule is explicitly written out
403(same typed ruled can of course be grouped together).
404
405* Pre and post actions.
406From: Florian Krohm <florian@edamail.fishkill.ibm.com>
407Subject: YYACT_EPILOGUE
408To: bug-bison@gnu.org
409X-Sent: 1 week, 4 days, 14 hours, 38 minutes, 11 seconds ago
410
411The other day I had the need for explicitly building the parse tree. I
412used %locations for that and defined YYLLOC_DEFAULT to call a function
413that returns the tree node for the production. Easy. But I also needed
414to assign the S-attribute to the tree node. That cannot be done in
415YYLLOC_DEFAULT, because it is invoked before the action is executed.
416The way I solved this was to define a macro YYACT_EPILOGUE that would
417be invoked after the action. For reasons of symmetry I also added
418YYACT_PROLOGUE. Although I had no use for that I can envision how it
419might come in handy for debugging purposes.
76551463 420All is needed is to add
69991a58
AD
421
422#if YYLSP_NEEDED
423 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen, yyloc, (yylsp - yylen));
424#else
425 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen);
426#endif
427
428at the proper place to bison.simple. Ditto for YYACT_PROLOGUE.
429
430I was wondering what you think about adding YYACT_PROLOGUE/EPILOGUE
431to bison. If you're interested, I'll work on a patch.
432
35fe0834
PE
433* Better graphics
434Equip the parser with a means to create the (visual) parse tree.
d7215705 435
f294a2c2
AD
436-----
437
401b73af
JD
438Copyright (C) 2001, 2002, 2003, 2004, 2006, 2008-2009 Free Software
439Foundation, Inc.
f294a2c2 440
51cbef6f 441This file is part of Bison, the GNU Compiler Compiler.
f294a2c2 442
f16b0819 443This program is free software: you can redistribute it and/or modify
f294a2c2 444it under the terms of the GNU General Public License as published by
f16b0819
PE
445the Free Software Foundation, either version 3 of the License, or
446(at your option) any later version.
f294a2c2 447
f16b0819 448This program is distributed in the hope that it will be useful,
f294a2c2
AD
449but WITHOUT ANY WARRANTY; without even the implied warranty of
450MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
451GNU General Public License for more details.
452
453You should have received a copy of the GNU General Public License
f16b0819 454along with this program. If not, see <http://www.gnu.org/licenses/>.