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