Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build fix for the newer bison environment #98

Open
GoogleCodeExporter opened this issue Feb 7, 2016 · 0 comments
Open

Build fix for the newer bison environment #98

GoogleCodeExporter opened this issue Feb 7, 2016 · 0 comments

Comments

@GoogleCodeExporter
Copy link

With the older bison, yyparse prototype is not contained in the generated 
header file[1].

However, the newer bison (I'm using 3.0.2 on Ubuntu 14.04) generates the header 
file with the yyparse prototype.
The yyparse prototype is declared with the parameter types. Since 
parser::PTXParser::State is nested class, we cannot add forward declaration 
into the generated header. And since PTXParser.h and yyparse prototype have a 
circular reference, we cannot include PTXParser.h in the generated 
ptxgrammar.hpp.

To solve it, I've created the patch for PTXLexer.h. In this patch, we provides 
yyparse macro definition to avoid generating yyparse prototype. I think it is 
relatively stable solution for the generated header by bison.

Could you review this patch?

[1]: https://lists.gnu.org/archive/html/bug-bison/2012-06/msg00013.html

Original issue reported on code.google.com by yusuke.suzuki@sslab.ics.keio.ac.jp on 1 Oct 2014 at 2:24

Attachments:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant