blob: 978b5c6f4d45fb3664a39e00cd06ec41051e5181 [file] [log] [blame]
Ying Wang05436632013-04-05 16:01:00 -07001* Short term
2** Graphviz display code thoughts
3The code for the --graph option is over two files: print_graph, and
4graphviz. I believe this is because Bison used to also produce VCG graphs,
5but since this is no longer true, maybe we could consider these files for
6fusion.
The Android Open Source Projectcea198a2009-03-03 19:29:17 -08007
Ying Wang05436632013-04-05 16:01:00 -07008Little effort factoring seems to have been given to factoring in these files,
9and their print-xml and print counterpart. We would very much like to re-use
10the pretty format of states from .output in the .dot
The Android Open Source Projectcea198a2009-03-03 19:29:17 -080011
Ying Wang05436632013-04-05 16:01:00 -070012Also, the underscore in print_graph.[ch] isn't very fitting considering
13the dashes in the other filenames.
14
15** Variable names.
16What should we name `variant' and `lex_symbol'?
17
18** Use b4_symbol in all the skeleton
19Move its definition in the more standard places and deploy it in other
20skeletons. Then remove the older system, including the tables
21generated by output.c
22
23** Update the documentation on gnu.org
24
25** Get rid of fake #lines [Bison: ...]
26Possibly as simple as checking whether the column number is nonnegative.
27
28I have seen messages like the following from GCC.
29
30<built-in>:0: fatal error: opening dependency file .deps/libltdl/argz.Tpo: No such file or directory
The Android Open Source Projectcea198a2009-03-03 19:29:17 -080031
32
Ying Wang05436632013-04-05 16:01:00 -070033** Discuss about %printer/%destroy in the case of C++.
34It would be very nice to provide the symbol classes with an operator<<
35and a destructor. Unfortunately the syntax we have chosen for
36%destroy and %printer make them hard to reuse. For instance, the user
37is invited to write something like
The Android Open Source Projectcea198a2009-03-03 19:29:17 -080038
Ying Wang05436632013-04-05 16:01:00 -070039 %printer { debug_stream() << $$; } <my_type>;
40
41which is hard to reuse elsewhere since it wants to use
42"debug_stream()" to find the stream to use. The same applies to
43%destroy: we told the user she could use the members of the Parser
44class in the printers/destructors, which is not good for an operator<<
45since it is no longer bound to a particular parser, it's just a
46(standalone symbol).
47
48** Rename LR0.cc
49as lr0.cc, why upper case?
50
51** bench several bisons.
52Enhance bench.pl with %b to run different bisons.
53
54* Various
55** Warnings
56Warnings about type tags that are used in printer and dtors, but not
57for symbols?
58
59** YYERRCODE
60Defined to 256, but not used, not documented. Probably the token
61number for the error token, which POSIX wants to be 256, but which
62Bison might renumber if the user used number 256. Keep fix and doc?
63Throw away?
64
65Also, why don't we output the token name of the error token in the
66output? It is explicitly skipped:
67
68 /* Skip error token and tokens without identifier. */
69 if (sym != errtoken && id)
70
71Of course there are issues with name spaces, but if we disable we have
72something which seems to be more simpler and more consistent instead
73of the special case YYERRCODE.
74
75 enum yytokentype {
76 error = 256,
77 // ...
78 };
The Android Open Source Projectcea198a2009-03-03 19:29:17 -080079
80
Ying Wang05436632013-04-05 16:01:00 -070081We could (should?) also treat the case of the undef_token, which is
82numbered 257 for yylex, and 2 internal. Both appear for instance in
83toknum:
The Android Open Source Projectcea198a2009-03-03 19:29:17 -080084
Ying Wang05436632013-04-05 16:01:00 -070085 const unsigned short int
86 parser::yytoken_number_[] =
87 {
88 0, 256, 257, 258, 259, 260, 261, 262, 263, 264,
The Android Open Source Projectcea198a2009-03-03 19:29:17 -080089
Ying Wang05436632013-04-05 16:01:00 -070090while here
91
92 enum yytokentype {
93 TOK_EOF = 0,
94 TOK_EQ = 258,
95
96so both 256 and 257 are "mysterious".
97
98 const char*
99 const parser::yytname_[] =
100 {
101 "\"end of command\"", "error", "$undefined", "\"=\"", "\"break\"",
102
103
104** YYFAIL
105It is seems to be *really* obsolete now, shall we remove it?
106
107** yychar == yyempty_
108The code in yyerrlab reads:
109
110 if (yychar <= YYEOF)
111 {
112 /* Return failure if at end of input. */
113 if (yychar == YYEOF)
114 YYABORT;
115 }
116
117There are only two yychar that can be <= YYEOF: YYEMPTY and YYEOF.
118But I can't produce the situation where yychar is YYEMPTY here, is it
119really possible? The test suite does not exercise this case.
120
121This shows that it would be interesting to manage to install skeleton
122coverage analysis to the test suite.
123
124** Table definitions
125It should be very easy to factor the definition of the various tables,
126including the separation bw declaration and definition. See for
127instance b4_table_define in lalr1.cc. This way, we could even factor
128C vs. C++ definitions.
129
130* From lalr1.cc to yacc.c
131** Single stack
132Merging the three stacks in lalr1.cc simplified the code, prompted for
133other improvements and also made it faster (probably because memory
134management is performed once instead of three times). I suggest that
135we do the same in yacc.c.
136
137** yysyntax_error
138The code bw glr.c and yacc.c is really alike, we can certainly factor
139some parts.
140
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800141
142* Report
143
Ying Wang05436632013-04-05 16:01:00 -0700144** Figures
145Some statistics about the grammar and the parser would be useful,
146especially when asking the user to send some information about the
147grammars she is working on. We should probably also include some
148information about the variables (I'm not sure for instance we even
149specify what LR variant was used).
150
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800151** GLR
152How would Paul like to display the conflicted actions? In particular,
Ying Wang05436632013-04-05 16:01:00 -0700153what when two reductions are possible on a given lookahead token, but one is
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800154part of $default. Should we make the two reductions explicit, or just
155keep $default? See the following point.
156
157** Disabled Reductions
158See `tests/conflicts.at (Defaulted Conflicted Reduction)', and decide
159what we want to do.
160
161** Documentation
162Extend with error productions. The hard part will probably be finding
163the right rule so that a single state does not exhibit too many yet
164undocumented ``features''. Maybe an empty action ought to be
165presented too. Shall we try to make a single grammar with all these
166features, or should we have several very small grammars?
167
168** --report=conflict-path
169Provide better assistance for understanding the conflicts by providing
170a sample text exhibiting the (LALR) ambiguity. See the paper from
171DeRemer and Penello: they already provide the algorithm.
172
Ying Wang05436632013-04-05 16:01:00 -0700173** Statically check for potential ambiguities in GLR grammars. See
174<http://www.i3s.unice.fr/~schmitz/papers.html#expamb> for an approach.
175
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800176
177* Extensions
178
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800179** $-1
180We should find a means to provide an access to values deep in the
181stack. For instance, instead of
182
Ying Wang05436632013-04-05 16:01:00 -0700183 baz: qux { $$ = $<foo>-1 + $<bar>0 + $1; }
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800184
185we should be able to have:
186
187 foo($foo) bar($bar) baz($bar): qux($qux) { $baz = $foo + $bar + $qux; }
188
189Or something like this.
190
191** %if and the like
192It should be possible to have %if/%else/%endif. The implementation is
193not clear: should it be lexical or syntactic. Vadim Maslow thinks it
194must be in the scanner: we must not parse what is in a switched off
195part of %if. Akim Demaille thinks it should be in the parser, so as
196to avoid falling into another CPP mistake.
197
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800198** XML Output
199There are couple of available extensions of Bison targeting some XML
200output. Some day we should consider including them. One issue is
201that they seem to be quite orthogonal to the parsing technique, and
202seem to depend mostly on the possibility to have some code triggered
203for each reduction. As a matter of fact, such hooks could also be
204used to generate the yydebug traces. Some generic scheme probably
205exists in there.
206
207XML output for GNU Bison and gcc
208 http://www.cs.may.ie/~jpower/Research/bisonXML/
209
210XML output for GNU Bison
211 http://yaxx.sourceforge.net/
212
213* Unit rules
214Maybe we could expand unit rules, i.e., transform
215
Ying Wang05436632013-04-05 16:01:00 -0700216 exp: arith | bool;
217 arith: exp '+' exp;
218 bool: exp '&' exp;
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800219
220into
221
Ying Wang05436632013-04-05 16:01:00 -0700222 exp: exp '+' exp | exp '&' exp;
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800223
224when there are no actions. This can significantly speed up some
225grammars. I can't find the papers. In particular the book `LR
226parsing: Theory and Practice' is impossible to find, but according to
227`Parsing Techniques: a Practical Guide', it includes information about
228this issue. Does anybody have it?
229
230
231
232* Documentation
233
234** History/Bibliography
235Some history of Bison and some bibliography would be most welcome.
236Are there any Texinfo standards for bibliography?
237
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800238* Coding system independence
239Paul notes:
240
Ying Wang05436632013-04-05 16:01:00 -0700241 Currently Bison assumes 8-bit bytes (i.e. that UCHAR_MAX is
242 255). It also assumes that the 8-bit character encoding is
243 the same for the invocation of 'bison' as it is for the
244 invocation of 'cc', but this is not necessarily true when
245 people run bison on an ASCII host and then use cc on an EBCDIC
246 host. I don't think these topics are worth our time
247 addressing (unless we find a gung-ho volunteer for EBCDIC or
248 PDP-10 ports :-) but they should probably be documented
249 somewhere.
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800250
Ying Wang05436632013-04-05 16:01:00 -0700251 More importantly, Bison does not currently allow NUL bytes in
252 tokens, either via escapes (e.g., "x\0y") or via a NUL byte in
253 the source code. This should get fixed.
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800254
255* --graph
256Show reductions.
257
258* Broken options ?
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800259** %token-table
260** Skeleton strategy
Ying Wang05436632013-04-05 16:01:00 -0700261Must we keep %token-table?
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800262
263* Precedence
264
265** Partial order
266It is unfortunate that there is a total order for precedence. It
267makes it impossible to have modular precedence information. We should
268move to partial orders (sounds like series/parallel orders to me).
269
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800270** RR conflicts
271See if we can use precedence between rules to solve RR conflicts. See
272what POSIX says.
273
274
275* $undefined
276From Hans:
277- If the Bison generated parser experiences an undefined number in the
278character range, that character is written out in diagnostic messages, an
279addition to the $undefined value.
280
281Suggest: Change the name $undefined to undefined; looks better in outputs.
282
283
284* Default Action
285From Hans:
286- For use with my C++ parser, I transported the "switch (yyn)" statement
287that Bison writes to the bison.simple skeleton file. This way, I can remove
288the current default rule $$ = $1 implementation, which causes a double
289assignment to $$ which may not be OK under C++, replacing it with a
290"default:" part within the switch statement.
291
292Note that the default rule $$ = $1, when typed, is perfectly OK under C,
293but in the C++ implementation I made, this rule is different from
294$<type_name>$ = $<type_name>1. I therefore think that one should implement
295a Bison option where every typed default rule is explicitly written out
296(same typed ruled can of course be grouped together).
297
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800298* Pre and post actions.
299From: Florian Krohm <florian@edamail.fishkill.ibm.com>
300Subject: YYACT_EPILOGUE
301To: bug-bison@gnu.org
302X-Sent: 1 week, 4 days, 14 hours, 38 minutes, 11 seconds ago
303
304The other day I had the need for explicitly building the parse tree. I
305used %locations for that and defined YYLLOC_DEFAULT to call a function
306that returns the tree node for the production. Easy. But I also needed
307to assign the S-attribute to the tree node. That cannot be done in
308YYLLOC_DEFAULT, because it is invoked before the action is executed.
309The way I solved this was to define a macro YYACT_EPILOGUE that would
310be invoked after the action. For reasons of symmetry I also added
311YYACT_PROLOGUE. Although I had no use for that I can envision how it
312might come in handy for debugging purposes.
313All is needed is to add
314
315#if YYLSP_NEEDED
316 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen, yyloc, (yylsp - yylen));
317#else
318 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen);
319#endif
320
321at the proper place to bison.simple. Ditto for YYACT_PROLOGUE.
322
323I was wondering what you think about adding YYACT_PROLOGUE/EPILOGUE
324to bison. If you're interested, I'll work on a patch.
325
Ying Wang05436632013-04-05 16:01:00 -0700326* Better graphics
327Equip the parser with a means to create the (visual) parse tree.
328
329* Complaint submessage indentation.
330We already have an implementation that works fairly well for named
331reference messages, but it would be nice to use it consistently for all
332submessages from Bison. For example, the "previous definition"
333submessage or the list of correct values for a %define variable might
334look better with indentation.
335
336However, the current implementation makes the assumption that the
337location printed on the first line is not usually much shorter than the
338locations printed on the submessage lines that follow. That assumption
339may not hold true as often for some kinds of submessages especially if
340we ever support multiple grammar files.
341
342Here's a proposal for how a new implementation might look:
343
344 http://lists.gnu.org/archive/html/bison-patches/2009-09/msg00086.html
345
346
347Local Variables:
348mode: outline
349coding: utf-8
350End:
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800351
352-----
353
Ying Wang05436632013-04-05 16:01:00 -0700354Copyright (C) 2001-2004, 2006, 2008-2012 Free Software Foundation, Inc.
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800355
356This file is part of Bison, the GNU Compiler Compiler.
357
Ying Wang05436632013-04-05 16:01:00 -0700358This program is free software: you can redistribute it and/or modify
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800359it under the terms of the GNU General Public License as published by
Ying Wang05436632013-04-05 16:01:00 -0700360the Free Software Foundation, either version 3 of the License, or
361(at your option) any later version.
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800362
Ying Wang05436632013-04-05 16:01:00 -0700363This program is distributed in the hope that it will be useful,
The Android Open Source Projectcea198a2009-03-03 19:29:17 -0800364but WITHOUT ANY WARRANTY; without even the implied warranty of
365MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
366GNU General Public License for more details.
367
368You should have received a copy of the GNU General Public License
Ying Wang05436632013-04-05 16:01:00 -0700369along with this program. If not, see <http://www.gnu.org/licenses/>.